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.
617 bytes, 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. — — 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: