Closed
Bug 259993
Opened 20 years ago
Closed 20 years ago
Messed up favicons on Bookmarks with Firefox 1.0 Preview Release
Categories
(Firefox :: Bookmarks & History, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: ghepeu, Assigned: vlad)
References
Details
(Keywords: fixed-aviary1.0)
Attachments
(2 files)
9.99 KB,
image/gif
|
Details | |
6.04 KB,
patch
|
bugs
:
review+
bugs
:
approval-aviary+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (X11; U; Linux i686; rv:1.7.3) Gecko/20040914 Firefox/0.10 Build Identifier: Mozilla/5.0 (X11; U; Linux i686; rv:1.7.3) Gecko/20040914 Firefox/0.10 When a site has no favicons, firefox displays other sites' favicons instead of "white sheet" icons. This problema occurs, for example, if I open in tabs all the bookmarks I have in a folder, and then open on the same window others site that haven't got favicons: site without favicon are associated to the favicon previously used in the same tab. For example: I open in tab all the 4 bookmarks in a folder so Firefox opens: tab 1: site 1 <=> favicon 1 tab 2: site 2 <=> favicon 2 tab 3: site 3 <=> favicon 3 tab 4: site 4 <=> favicon 4 then, I open another bookmarks folder with 3 items, so I get: tab 1: site A <=> favicon A tab 2: site B <=> without favicon tab 3: site C <=> without favicon tab 4: closed then on Bookmarks I have: site 1 <=> favicon 1 site 2 <=> favicon 2 site 3 <=> favicon 3 site 4 <=> favicon 4 site A <=> favicon A site B <=> favicon 2 (instead of white sheet icon) site C <=> favicon 3 (instead of white sheet icon) Reproducible: Always Steps to Reproduce:
Comment 1•20 years ago
|
||
*** Bug 260043 has been marked as a duplicate of this bug. ***
Comment 2•20 years ago
|
||
Yep I'm getting the same problem. What's more annoying is that on occasion Firefox will replace a bookmark correct icon with an incorrect icon and there seems to be no way to revert to the old one. I have to backup my bookmarks file and restore it a lot to prevent this from happening.
Same issue as Morac. In the screenshot, the first bookmark has the IMDB favicon for some reason, and the second bookmark has the favicon of the third. This is replicated in many places in my bookmarks. FF 1.0 PR on WinXP SP2.
Comment 4•20 years ago
|
||
I imagine it's really frustrating for the devs to have to deal with so many bugs over such a little thing as favicons. That said, it is a *very* visible feature, that for 1.0 really needs to "just work" IMHO. I've been bitten by this one several times, and the only fix is to delete and recreate the bookmark. It's pretty annoying, so nominating to block 1.0 accordingly.
Flags: blocking-aviary1.0?
Assignee | ||
Comment 5•20 years ago
|
||
Fix. The root of the bug comes from us not being able to adequately recognize when new content is being loaded in a browser view other than the current (in the progress listener), because of tabbrowser/browser split. This is a semi-bandaid; I'm hoping that eventually we can merge tabbrowser/browser and have all these problems go away (i.e. have the progress listener be told explicitly which sub-browser's progress is being changed, instead of it assuming "the current one").
Assignee | ||
Updated•20 years ago
|
Attachment #160193 -
Flags: review?(bugs)
Comment 6•20 years ago
|
||
Comment on attachment 160193 [details] [diff] [review] 259993-favicon-open-tabs-fix-0.patch r+a=ben@mozilla.org
Attachment #160193 -
Flags: review?(bugs)
Attachment #160193 -
Flags: review+
Attachment #160193 -
Flags: approval-aviary+
Assignee | ||
Comment 7•20 years ago
|
||
in on aviary.
Status: NEW → RESOLVED
Closed: 20 years ago
Flags: blocking-aviary1.0?
Keywords: fixed-aviary1.0
Resolution: --- → FIXED
I believe this may have caused Bug 262272.
Comment 9•20 years ago
|
||
I still have this problem with the latest version
Comment 10•20 years ago
|
||
If anyone is still suffering from this bug, and your bookmarks have messed up favicons, it means you have to export your bookmark file and manually remove all of the icon="*whatever*" references from it. Restore the fixed bookmark file, and you should be fine. :)
Comment 11•20 years ago
|
||
I guess we should move to yet another new bug (maybe one for the tabbrowser/browser merge work that Vlad mentioned?) I can't make it happen reliably, but using Firefox 1.0 release and starting with a new bookmark file, and then repeatedly messing with loading the BBC news main page (http://news.bbc.co.uk/) and other sites (e.g. MozillaZine) in tabs simultaneously, I can still get the wrong icon onto the BBC bookmark. This "bandaid" fix has improved things, but the problem still exists.
Comment 12•20 years ago
|
||
For the record, bug 271359 is open for residual issues not fixed by the band-aid fix done here
Comment 13•18 years ago
|
||
sorry for bugspam, long-overdue mass reassign of ancient QA contact bugs, filter on "beltznerLovesGoats" to get rid of this mass change
QA Contact: mconnor → bookmarks
You need to log in
before you can comment on or make changes to this bug.
Description
•