1204513 - DOM Full Screen should be really full screen

Status: RESOLVED FIXED

(SeaMonkey :: UI Design, defect)

Description

[[e-mail]]

Normal F11 full screen doesn't hide the navigation bar or tabbar by design. However DOM full screen should hide everything. We can detect DOM full screen by looking at the mozFullScreen property on the document.

Note that this bug is not about turning on DOM full screen by default, as there are various warnings and permissions that should be implemented first.

Comment 1

[[e-mail]]

Attachment: Proposed patch

1. Hide the tab bar when DOM full screen is active
2. The navbar is not special when entering DOM full screen
3. Ensure the navbar is hidden when exiting full screen

Comment 2

[Philip Chee]

Comment on attachment 8660746 [details] [diff] [review]
Proposed patch

In Normal mode:
> 1. Hide the tab bar when DOM full screen is active
Tab bar is hidden
> 2. The navbar is not special when entering DOM full screen
Nav bar is hidden
> 3. Ensure the navbar is visible when exiting full screen
Nav bar is visible when exiting DOM full screen

All chrome is hidden in DOM full screen mode. There is a thin strip of something across the top. It's not the full screen toggler since we don't have one.
Normal UI restored on exiting DOM full screen.

In F11 full screen mode:
> 1. Hide the tab bar when DOM full screen is active
> 2. The navbar is not special when entering DOM full screen
Tab bar and Nav bar remain visible on entering DOM full screen.
> 3. Ensure the navbar is visible when exiting DOM full screen
Nav bar is visible when exiting DOM full screen

Comment 3

[Philip Chee]

TODO? DOM full screen should:
1. hide findbar if open
2. hide notification box
3. hide sidebar

Comment 4

[[e-mail]]

Attachment: Hide more stuff

Comment 5

[[e-mail]]

Comment on attachment 8662050 [details] [diff] [review]
Hide more stuff

Bah, Bugzilla ate my reviewer :-( [I blame the autocomplete.]

Comment 6

[Philip Chee]

Comment on attachment 8662050 [details] [diff] [review]
Hide more stuff

Sorry for the delay r+

Comment 7

[[e-mail]]

Pushed comm-central changeset 884d1267eff3.