Returning from fullscreenmode with minimized toolbar, bookmark-bar and menu-bar produces 5 instead of 3 "grip" buttons

RESOLVED FIXED in seamonkey2.6

Status

SeaMonkey
UI Design
--
minor
RESOLVED FIXED
10 years ago
6 years ago

People

(Reporter: Christoph, Assigned: neil@parkwaycc.co.uk)

Tracking

Trunk
seamonkey2.6

SeaMonkey Tracking Flags

(seamonkey2.4 wontfix, seamonkey2.5 fixed, seamonkey2.6 fixed)

Details

(Whiteboard: [SmBugEvent])

Attachments

(2 attachments)

(Reporter)

Description

10 years ago
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.8) Gecko/20071009 SeaMonkey/1.1.5
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.8) Gecko/20071009 SeaMonkey/1.1.5

Switching to fullscreen via `F11' with all the UI elements minimized results later  switching back again in more maximize buttons (in total 5).

Reproducible: Always

Steps to Reproduce:
1. Minimize all UI elements (navigation toolbar, menubar, bookmark ...)
2. Switch to fullscreen and back
Actual Results:  
Observed more maximize buttons.

Expected Results:  
Show just the three buttons like before switching to fullscreen

I am using the classic theme. Vanilla build, directly downlaoded from the homepage. Present extensions: flashplayer, adblock plus. All with vanilla config.
Confirmed on trunk:

* The navigation toolbar behaves correctly
* When returning to normal mode, the menubar and site navigation bar are expanded, and minimised grippies displayed as well.
Status: UNCONFIRMED → NEW
Ever confirmed: true
[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9a5pre) Gecko/20070515 SeaMonkey/1.5a] (nightly) (W2Ksp4)
[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9pre) Gecko/2008061402 SeaMonkey/2.0a1pre] (nightly) (W2Ksp4)

Confirming (comment 1):
*no extensions
*both themes

Affected:
Menu Bar
Personal Toolbar
Site Navigation Bar [which has bug 439312 too]

Working:
Navigation Toolbar
Severity: trivial → minor
Version: unspecified → Trunk
Mozilla/5.0 (X11; Linux x86_64; rv:2.0.2pre) Gecko/20110517 Firefox/4.0.2pre SeaMonkey/2.1.1pre
no extensions (other than the built-in ones)
tested only with default theme

Setting Platform to All.

On Linux, I see four grippies on return from full-screen mode, instead of the "expected" three. The extra grippy comes from the menubar (collapsing the menubar by means of its grippy, then hitting F11 twice, produces 2 grippies in collapsed position). Behaviour of the other two bars is OK. The two menubar grippies both say "Menu Bar" when hovered in collapsed state, but after expanding one of them, clicking the other simply makes it disappear.

Tentatively moving this bug to UI Design. Neil, if I goofed, go ahead and move it where it belongs.
Assignee: general → nobody
Component: General → UI Design
OS: Windows XP → All
QA Contact: general → ui-design
Hardware: x86 → All
Whiteboard: [SmBugEvent]
P.S. when hitting F11 repeatedly with the menubar collapsed, I see one additional grippy every time I come back out of fullscreen. If the other toolbars are also collapsed they don't exhibit such behaviour.

Comment 5

7 years ago
Having the File menu bar collapsed seem to be the precipitater to this.
(I.e. having the other menu bars, other then File, closed does not cause this to occur.)

Mozilla/5.0 (Windows NT 5.1; rv:2.0.1) Gecko/20110511 Firefox/4.0.1 SeaMonkey/2.1


This does not occur in SeaMonkey 2.0.

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1.19) Gecko/20110420 SeaMonkey/2.0.14
(Assignee)

Comment 6

6 years ago
Created attachment 553167 [details] [diff] [review]
Minimal patch [Checkin: comments 8 and 10]

Going in or out of full screen changes the menubar's binding, so when we exit full screen we try to create the collapsed grippy again.
Assignee: nobody → neil
Status: NEW → ASSIGNED
Attachment #553167 - Flags: review?(iann_bugzilla)
(Assignee)

Comment 7

6 years ago
Created attachment 553171 [details] [diff] [review]
Possible patch

Some of the grippy code looked like it could use some cleanup.

Updated

6 years ago
Attachment #553167 - Flags: review?(iann_bugzilla) → review+
(Assignee)

Comment 8

6 years ago
Pushed changeset 33dd068ac121 to comm-central.
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
(Assignee)

Comment 9

6 years ago
Comment on attachment 553167 [details] [diff] [review]
Minimal patch [Checkin: comments 8 and 10]

Probably not significant enough a regression for beta so just trying aurora.
Attachment #553167 - Flags: approval-comm-aurora?

Updated

6 years ago
Attachment #553167 - Flags: approval-comm-aurora? → approval-comm-aurora+
(Assignee)

Updated

6 years ago
Keywords: checkin-needed
Whiteboard: [SmBugEvent] → [transplant changeset 33dd068ac121 to comm-aurora][SmBugEvent]
Comment on attachment 553167 [details] [diff] [review]
Minimal patch [Checkin: comments 8 and 10]

http://hg.mozilla.org/releases/comm-aurora/rev/2c601ad0b4d0
Attachment #553167 - Attachment description: Minimal patch → Minimal patch [Checkin: comments 8 and 10]
status-seamonkey2.4: --- → wontfix
status-seamonkey2.5: --- → fixed
status-seamonkey2.6: --- → fixed
Keywords: checkin-needed
Whiteboard: [transplant changeset 33dd068ac121 to comm-aurora][SmBugEvent] → [SmBugEvent]
Target Milestone: --- → seamonkey2.6
You need to log in before you can comment on or make changes to this bug.