Closed Bug 262566 Opened 20 years ago Closed 20 years ago

Print preview should hide the browser statusbar

Categories

(Core :: Print Preview, enhancement)

Product:
Core
Component:
Print Preview
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: Stefan.Borggraefe, Assigned: Stefan.Borggraefe)

Details

Attachments

(1 file, 1 obsolete file)

New Patch
2.55 KB, patch
timeless
: review+
neil
: superreview+
Details | Diff | Splinter Review 
I think Firefox bug 259005 makes sense for Seamonkey, too. In Seamonkey you also
have the icons to switch to the different application parts (MailNews, CZ, ...),
but this isn't exactly what you typically want to do from Print Preview.
Attached patch Stolen from bug 259005 ;-) (obsolete) — — Splinter Review 

    
  

        Attachment #160818 -
        Flags: superreview?(neil.parkwaycc.co.uk)

        Attachment #160818 -
        Flags: review?(mconnor)

    
    

    
  
Comment on attachment 160818 [details] [diff] [review]
Stolen from bug 259005 ;-)

yeah, it just doesn't make sense with our current print preview impl.

        Attachment #160818 -
        Flags: review?(mconnor) → review+

    
    

    
  
Comment on attachment 160818 [details] [diff] [review]
Stolen from bug 259005 ;-)

If you hide the status bar from the menu (not an option in firefox?) then it
reappears when exiting print preview.

        Attachment #160818 -
        Flags: superreview?(neil.parkwaycc.co.uk) → superreview-

    
    

    
  

      
      
      
      

        Attached patch
          
          
        New Patch
        — 
          — Splinter Review
      

    


  
Firefox also has the "hide status bar" feature and the patch I initially ported
from Firefox introduced the problem Neil spotted into Firefox. I reported bug
263599 about it.

This is a new patch for Seamonkey without this flaw.

        Attachment #160818 -
        Attachment is obsolete: true

    
    

    
  
Comment on attachment 162380 [details] [diff] [review]
New Patch

timeless, could you review this patch? mconnor seems to be not available
currently.

        Attachment #162380 -
        Flags: review?(timeless)

    
  

        Attachment #162380 -
        Flags: superreview?(neil.parkwaycc.co.uk)

        Attachment #162380 -
        Flags: review?(timeless)

        Attachment #162380 -
        Flags: review+

    
  

        Attachment #162380 -
        Flags: superreview?(neil.parkwaycc.co.uk) → superreview+

    
    

    
  
Fix checked in.
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED

          You need to log in
          before you can comment on or make changes to this bug.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: