Skip to content

Commit 0980912

Browse files
committed
fix: bookmarks: incorrect recalc of folder lengths on bookmark rm/move
Calculate only 'bookmark' type
1 parent de923dc commit 0980912

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/services/bookmarks.handlers.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -123,7 +123,7 @@ function onBookmarkMovedFg(id: ID, info: browser.bookmarks.MoveInfo): void {
123123
// Update length of parent folders
124124
const node = Bookmarks.reactive.byId[id]
125125
if (node && oldParent && newParent && newParent.id !== oldParent.id) {
126-
const movedLen = node?.len || 1
126+
const movedLen = node?.len || (node.type === 'bookmark' ? 1 : 0)
127127
Bookmarks.updateTreeLen(oldParent, -movedLen)
128128
Bookmarks.updateTreeLen(newParent, movedLen)
129129

@@ -160,7 +160,7 @@ function onBookmarkRemovedFg(id: ID, info: browser.bookmarks.RemoveInfo): void {
160160
if (!node) return
161161

162162
// Update length of parent folders
163-
const removedLen = node.len || 1
163+
const removedLen = node.len || (node.type === 'bookmark' ? 1 : 0)
164164
Bookmarks.updateTreeLen(parent, -removedLen)
165165

166166
// Remove from tree

0 commit comments

Comments
 (0)