-
-
Notifications
You must be signed in to change notification settings - Fork 45
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Feature Request] Disable and/or on-demand reset of cached favicons #492
Comments
I'd like to better-understand your performance issue:
It's very unlikely that favicons are the cause of any issues on browser startup or page load—they are not loaded at all on browser startup. The only time they are loaded is when the Tab Stash UI is opened, at which point they are loaded in the background after the rest of the UI is ready (so generally will not affect responsiveness). |
115.13.0esr (64-bit) Not the best choice of words , by that i mean the delay it takes for firefox to enter "idle" after startup with the extension enabled, theres no impact on browsing The large places.sqlite is the cause as the delay does happen with other bookmark related extensions aswell when opening their internal pages |
120k bookmarks is going to be the reason for the slow startup/loading times. Currently Tab Stash loads the entire bookmarks database into memory when your browser starts, and also when opening the Tab Stash UI (which cannot share its data with the background page, unfortunately). It does this because that's how we keep up to date with folders/sub-trees being moved/deleted, etc. especially things being moved into and out of the stash. There are more intelligent ways to do this, I just haven't had time to implement them yet. (That work will be done as part of #97.)
Yes, right now we cache favicons for open tabs, so that if you stash that tab, we can keep its icon. This cache is cleared of stale things periodically, but if you have an open tab or saved bookmark, that's enough to keep an icon in the cache. But, again, this is unlikely to be the cause of responsiveness issues on browser startup or on loading the Tab Stash UI. You could have millions of icons in there and the worst case would be wasted space—they're added to the cache eagerly, but only loaded on demand (e.g. opening the UI), and even then in the background. |
Problem Statement
Lots of open tabs/(stash/regular)bookmarks ; visible impact on performance with extension enabled , e.g. on browser startup and opening the stash page
Preferred Solution(s)
Option to disable any favicon operations , can decrease ram/cpu usage and save disk space not relying on a cache database to fuction . Extension could use a generic icon instead for tabs and stash
(Tried to lock the databse with a empty file but that prevents the extension to function properly, unable to collapse open/unstashed tabs section or expand the stash groups , did not try to stash anything though 😄 )
Alternative Solution(s)
Option to disable favicons operations only for the stashed tabs (bookmarks) and able to clear the database on demand, Will still allow caching of favicons for open/unstashed tabs.
Additional Context
Firefox 115.12.0esr (64-bit)
Tab Stash 3.0.1
Readers: If you are also interested in seeing this feature be developed, please vote for it by giving the ORIGINAL POST a thumbs-up using the 😃 button. You are welcome to leave comments and discuss the feature request, but "Me too!" comments are not counted by the voting system.
The text was updated successfully, but these errors were encountered: