Closed
Bug 540629
Opened 16 years ago
Closed 16 years ago
Use superior autohiding menubar in full screen mode?
Categories
(SeaMonkey :: UI Design, enhancement)
SeaMonkey
UI Design
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: neil, Assigned: neil)
References
(Blocks 1 open bug, )
Details
Attachments
(1 file)
|
1.68 KB,
patch
|
iannbugzilla
:
review+
|
Details | Diff | Splinter Review |
Currently we just blindly hide the menubar in full screen mode, which makes the menus inaccessible. We should autohide it instead.
|
Assignee | |
Comment 1•16 years ago
|
||
I had to use the autohide attribute because the toolkit binding uses it to check whether it is still applicable, but I simplified the CSS rules since we always want to autohide the menubar in full screen mode even if it has been customised.
|
||
Comment 2•16 years ago
|
||
> +#toolbar-menubar[autohide="true"]
No grippy in Full screen mode? I guess that makes sense.
(In reply to comment #1)
> Created an attachment (id=422343) [details]
> Proposed patch
>
> I had to use the autohide attribute because the toolkit binding uses it to
> check whether it is still applicable, but I simplified the CSS rules since we
> always want to autohide the menubar in full screen mode even if it has been
> customised.
I cannot spot any difference when testing this on linux, would be expected to see any?
|
|
Assignee | |
Comment 4•16 years ago
|
||
Whoops, it would help if I explained what autohide was, but maybe it would be easier just to demonstrate: just press (say) Alt+V in full screen mode.
Attachment #422343 -
Flags: review?(iann_bugzilla) → review+
|
|
Assignee | |
Comment 5•16 years ago
|
||
Pushed changeset 3d183cb5d855 to comm-central.
|
|
Assignee | |
Comment 6•16 years ago
|
||
Whoops, forgot to resolve this as fixed.
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
|
|
||
Updated•12 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•