Closed Bug 120949 Opened 23 years ago Closed 23 years ago

Disable favicons (for 0.9.8 and probably 1.0)

Categories

(SeaMonkey :: General, defect)

Product:
SeaMonkey
Component:
General
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
mozilla0.9.8

People

(Reporter: hyatt, Assigned: hyatt)

References

(Blocks 1 open bug)

Details

(Keywords: relnote)

Attachments

(1 file)

Patch.
(deleted), patch
pavlov
: review+
bugzilla
: superreview+
Details | Diff | Splinter Review
I believe favicons should be disabled for 1.0. The open issues with them have grown large enough that 2 milestones (when I have to devote my time to many more important features like perf/stability/XUL1.0/XBL1.0) won't cut it. This bug is about turning the pref off for favicons in Mozilla. Post 1.0, we can revisit the favicons problem.
Status: NEW → ASSIGNED
Summary: Disable favicons for 1.0 → Disable favicons (for 0.9.8 and probably 1.0)
Target Milestone: --- → mozilla0.9.8
Attached patch Patch. (deleted) — — Splinter Review
Comment on attachment 65774 [details] [diff] [review] Patch. r=pavlov
Attachment #65774 - Flags: review+
Comment on attachment 65774 [details] [diff] [review] Patch. sr=blake nice last minute attempt to save your job.
Attachment #65774 - Flags: superreview+
Heh. ;)
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
OS: Windows 2000 → All
Hardware: PC → All
Asa: Can we relnote this one ? I bet we will get more than 30bugs in the first week about not working favicons.
Keywords: relnote
Way to get drivers' a= before checkin! /be
Blocks: 115520
Blocks: 120352
Changing summary to make it clear that this change reversed bug 109843.
Summary: Disable favicons (for 0.9.8 and probably 1.0) → Turn off aggressive favicon.ico search (for 0.9.8 and probably 1.0)
Please stop using the bug system to editorialize.
Summary: Turn off aggressive favicon.ico search (for 0.9.8 and probably 1.0) → Disable favicons (for 0.9.8 and probably 1.0)
jrgm: This bug's current summary, the same summary I tried to edit, makes users think that site icons have been completely disabled. I am not simply speculating; see Matti's comment from this morning in bug 121830. Why is it "editorializing" to change a misleading summary?
The first thing I noticed after upgrading 0.97 to 0.98 were the missing favicons (at least from that pages that use the "IE-Style"). Please try to make them work again at least for 1.0 THX Thorsten
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: