NeilBrommer/NewTabPage
1
0
Fork 0
Code Issues Pull requests Packages Projects Releases Wiki Activity

Fix bug when moving groups up the list

Browse source
This commit is contained in:
Neil Brommer 2017-12-28 13:41:36 -08:00
parent 4a5a75cfa8
commit abed596f79
1 changed files with 13 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
js/editBookmarks.js
View file

@ -62,12 +62,21 @@ function groupMoved(dropEvt) {
groupsStore.getAll().onsuccess = function (evt) { groupsStore.getAll().onsuccess = function (evt) {
var groups = evt.target.result; var groups = evt.target.result;
if (newIndex > oldIndex) {
for (let g of groups) { for (let g of groups) {
if (g.groupIndex > oldIndex && g.groupIndex <= newIndex) { if (g.groupIndex > oldIndex && g.groupIndex <= newIndex) {
g.groupIndex--; g.groupIndex--;
groupsStore.put(g); groupsStore.put(g);
} }
} }
} else { // oldIndex > newIndex
for (let g of groups) {
if (g.groupIndex < oldIndex && g.groupIndex >= newIndex) {
g.groupIndex++;
groupsStore.put(g);
}
}
}
groups[oldIndex].groupIndex = newIndex; groups[oldIndex].groupIndex = newIndex;
groupsStore.put(groups[oldIndex]); groupsStore.put(groups[oldIndex]);