Closed Bug 401552 Opened 18 years ago Closed 14 years ago

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

Categories

(SeaMonkey :: UI Design, defect)

defect
Not set
minor

Tracking

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

RESOLVED FIXED
seamonkey2.6
Tracking Status
seamonkey2.4 --- wontfix
seamonkey2.5 --- fixed
seamonkey2.6 --- fixed

People

(Reporter: loszivos, Assigned: neil)

Details

(Whiteboard: [SmBugEvent])

Attachments

(2 files)

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.
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
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)
Attached patch Possible patchSplinter Review
Some of the grippy code looked like it could use some cleanup.
Attachment #553167 - Flags: review?(iann_bugzilla) → review+
Pushed changeset 33dd068ac121 to comm-central.
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
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?
Attachment #553167 - Flags: approval-comm-aurora? → approval-comm-aurora+
Keywords: checkin-needed
Whiteboard: [SmBugEvent] → [transplant changeset 33dd068ac121 to comm-aurora][SmBugEvent]
Attachment #553167 - Attachment description: Minimal patch → Minimal patch [Checkin: comments 8 and 10]
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.

Attachment

General

Created:
Updated:
Size: