rowan/multi-account-containers
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fixed if no container is selected, shift shouldn't work issue

Browse source
This commit is contained in:
joey 2020-03-09 13:19:00 -04:00
parent f82fd5f519
commit ed7698f458
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/js/popup.js
View file

@ -1064,7 +1064,7 @@ Logic.registerPanel(P_CONTAINERS_EDIT, {
const currentSelectedIdentity = Logic.currentSelectedIdentities(); const currentSelectedIdentity = Logic.currentSelectedIdentities();
const index = currentSelectedIdentity.indexOf(identity); const index = currentSelectedIdentity.indexOf(identity);
if (!this.lastSelected || this.shiftOn === 0) { if (!this.lastSelected || this.shiftOn === 0 || currentSelectedIdentity.length === 0) {
this.lastSelected = identity; this.lastSelected = identity;
} }
@ -1321,7 +1321,7 @@ Logic.registerPanel(P_CONTAINER_DELETE, {
// right now for mult-selection, it displays the first item in the selection at the icon and name // right now for mult-selection, it displays the first item in the selection at the icon and name
// Populating the panel: name, icon, and warning message // Populating the panel: name, icon, and warning message
document.getElementById("delete-container-name").textContent = currentSelection[0].name; document.getElementById("delete-container-name").textContent = currentSelection[0].name;
document.getElementById("delete-container-tab-warning").textContent = ``; document.getElementById("delete-container-tab-warning").textContent = "";
for (let i = 0; i < currentSelection.length; i++) { for (let i = 0; i < currentSelection.length; i++) {
const identity = currentSelection[i]; const identity = currentSelection[i];
const totalNumberOfTabs = identity.numberOfHiddenTabs + identity.numberOfOpenTabs; const totalNumberOfTabs = identity.numberOfHiddenTabs + identity.numberOfOpenTabs;