465020 - Toolbars are not hidden in Full Screen and Print Preview after Bug 428216

Status: RESOLVED FIXED

(SeaMonkey :: UI Design, defect)

Description

[Philip Chee]

+++ This bug was initially created as a clone of Bug #428216 +++

[From Bug 428216 Comment 68]
>> Also at least for me the toolbars do not change when activating Print Preview
>> from a browser window (i.e. the browser toolbars stay while the content area
>> switches to Print Preview).

> And neither does full screen F11. This was working in an earlier patch and I
> forgot to test for this in the latest patch. I'll file a bug (Sigh)

Comment 1

[Philip Chee]

Attachment: Fullscreen Fix 1.0

1. When I removed the smart getter/setter I forgot to restore the global gNavToolbox variable.

2. Also moved code that is global from showXULChrome() to toggle() since there is no need to invoke these lines thrice.

Comment 2

[Robert Kaiser]

pushed as http://hg.mozilla.org/comm-central/rev/837c86f7e3e6 - Philip, please mark fixed if everything works with this.

Comment 3

[Philip Chee]

Resolved FIXED

Comment 4

[Tony Mechelynck [:tonymec]]

Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1b2pre) Gecko/20081122 SeaMonkey/2.0a2pre - Build ID: 20081122000456

"Verified" for Linux.

No browser toolbars in Print Preview (of course, the Page Setup toolbar is displayed at top, which is correct). In F11 "full screen" mode the menubar, bookmarks bar and Site-navigation bar all disappear, as well as the statusbar, leaving only the bar with the URL bar (which moves up to where the titlebar used to be), and the tabbar.