victrme / Bonjourr

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

When clicking on a Quick Link, browser request for images.unsplash.com always appears #323

Open Gitoffthelawn opened 5 months ago

Gitoffthelawn commented 5 months ago

Describe the bug When clicking on any Quick Link, a browser request for images.unsplash.com always appears in the browser's status area (the area, typically in the lower-left corner of the browser window, that momentarily displays the URL being loaded).

To Reproduce Steps to reproduce the behavior:

  1. Set up a Quick Link, in particular one that takes a little time to load (e.g. a slow server).
  2. Click on that Quick link.
  3. Notice that the message "Transferring data from images.unsplash.com" appears in Firefox's status area.

Expected behavior No data transferred to/from unsplash.com unless the Quick Link is to unsplash.com

Device:

victrme commented 5 months ago

I believe this is a UI bug in Firefox, not an issue with Bonjourr.

What seems to be happening is that Firefox detects a request, displays the request status, and because [insert Firefox bug here], the status bar still shows the last requested URL.

There is nothing in the network tab that shows a request to images.unsplash.com. Also the message lasts as long as the server is unreachable, despite the image being already cached.

Like the previous issue, I'll leave this open. I might be missing something !

Gitoffthelawn commented 5 months ago

Thanks. I think you are likely correct in your conclusion.

Gitoffthelawn commented 5 months ago

@victrme Would you be willing to report this to Mozilla on Bugzilla? I feel like you may be able to provide more details as one of the authors of Bonjourr.

victrme commented 5 months ago

I only got the issue twice and can't reproduce it... I think it needs to be investigated a bit more to make sure this is actually a Firefox UI problem before reporting.

Right now I don't have time to investigate though, so I'll re-open the issue for now.