victrme / Bonjourr

Minimalist & lightweight startpage inspired by iOS
https://bonjourr.fr
GNU General Public License v3.0
998 stars 115 forks source link

I recommend caching the icon for quick links to prevent occasional load times when opening the browser #414

Closed yin2hao closed 3 months ago

yin2hao commented 3 months ago

I recommend caching the icon for quick links to prevent occasional load times when opening the browser

victrme commented 3 months ago

Icons are currently cached using the browser memory and disk cache, but are not stored locally. I tried to implement a cache with indexedDB, but the page is loading much faster than indexedDB can fetch the icons, resulting in links icons flashing on every new tabs.

I could try something with service workers now that I think about it 🤔

Gitoffthelawn commented 3 months ago

If you choose to implement this, please don't forget that some users don't show quick link icons at all. In these cases, no icon service workers should be invoked.