stonewareslord/treestyletab
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Don't activate UI to manage tree relations for bookmarks in the "unsorted bookmarks" folder.

Browse Source 
We often store very large number of items into the folder, however we'll never open them as a group of bookmarks via middle click on the folder. On the other hand, if there are too many sibling bookmarks, TST can freeze to initialize the list of siblings. So I decided to disable the UI for bookmarks in the "unsorted bookmarks" folder. This decision is related to the issue #398.
This commit is contained in:
Piro / YUKI Hiroshi 2013-10-30 01:40:20 +09:00
parent 6756ec255a
commit 63114ca99f
1 changed files with 6 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
content/treestyletab/bookmarksOverlayEditable.js
View File

@ -153,6 +153,12 @@ var TreeStyleTabBookmarksServiceEditable = {
this.canceled = false; this.canceled = false;
// Ignore bookmark in the "unsorted bookmarks" folder, because
// there can be very large number of bookmarks and they won't be
// opened as a tree.
if (PlacesUtils.bookmarks.getFolderIdForItem(id) == PlacesUtils.unfiledBookmarksFolderId)
return;
this._createSiblingsFragment(id, (function(aFragment) { this._createSiblingsFragment(id, (function(aFragment) {
var siblings = Array.slice(aFragment.childNodes) var siblings = Array.slice(aFragment.childNodes)
.map(function(aItem) { .map(function(aItem) {