Closed
Bug 983632
Opened 10 years ago
Closed 10 years ago
[Australis, Windows HiDPI] button icons can be completely hidden but still be on toolbar and not in overflow menu if window resized/width decreased
Categories
(Firefox :: Toolbars and Customization, defect)
Tracking
()
VERIFIED
FIXED
Firefox 30
People
(Reporter: aryx, Assigned: jaws)
References
(Blocks 1 open bug)
Details
(Whiteboard: [Australis:P3])
Attachments
(2 files)
3.99 KB,
image/png
|
Details | |
1.06 KB,
patch
|
Gijs
:
review+
Sylvestre
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
Nightly and Aurora 20140313 on Windows 8.1 Pro (OS level font size 125%) When the browser window's width gets decreased, it is possible that the icon of a button gets completely hidden, but some of clickable area is still available (see screenshot). In my humble opinion, the button should get added to the overflow. E.g. after the icon getting partially hidden, or at a threshold (like 50%).
Comment 1•10 years ago
|
||
Can you provide STR? On OS X and Linux, we have a recent regression bug filed, where the QA person who reported it said it's not reproducible on Windows, so...
Flags: needinfo?(archaeopteryx)
![]() |
Reporter | |
Comment 3•10 years ago
|
||
(In reply to :Gijs Kruitbosch from comment #1) > Can you provide STR? 1. Create new profile and launch it (window opens not maximized). 2. Reduce window width.
Flags: needinfo?(archaeopteryx)
Comment 4•10 years ago
|
||
Sigh.
Summary: [Australis] button icons can be completely hidden but still be on toolbar and not in overflow menu if window resized/width decreased → [Australis, Windows HiDPI] button icons can be completely hidden but still be on toolbar and not in overflow menu if window resized/width decreased
Comment 5•10 years ago
|
||
I expect this is a core bug with overflow events, considering that I can't reproduce on 100% DPI but can on 125% (and Jared reproduced on 200% DPI).
Whiteboard: [Australis:P3]
Assignee | ||
Comment 7•10 years ago
|
||
I looked for the regression range and this has been present since Australis merged to mozilla-central. http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=beddd6d4bcdf&tochange=ba9ecdea3a90 I think this has something to do with overflowing at a slow pace, because that's the only way that I can reproduce it. My guess is that our calculations for determining if something is overflowing don't trigger if it's only a couple pixels, whereas if you resize the window quickly then this bug doesn't appear.
Assignee | ||
Comment 8•10 years ago
|
||
Attachment #8391240 -
Flags: review?(gijskruitbosch+bugs)
Assignee | ||
Updated•10 years ago
|
Status: NEW → ASSIGNED
Updated•10 years ago
|
Attachment #8391240 -
Flags: review?(gijskruitbosch+bugs) → review+
Assignee | ||
Comment 9•10 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/d7e5a58a314d
Whiteboard: [Australis:P3] → [Australis:P3][fixed-in-fx-team]
https://hg.mozilla.org/mozilla-central/rev/d7e5a58a314d
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [Australis:P3][fixed-in-fx-team] → [Australis:P3]
Target Milestone: --- → Firefox 30
Comment 12•10 years ago
|
||
Comment on attachment 8391240 [details] [diff] [review] Patch [Approval Request Comment] Bug caused by (feature/regressing bug #): Australis' overflowable toolbar User impact if declined: sometimes the toolbar doesn't overflow correctly because we don't get overflow events Testing completed (on m-c, etc.): on m-c Risk to taking this patch (and alternatives if risky): very low String or IDL/UUID changes made by this patch: none NB: uplifting this is important if we uplift bug 980879.
Attachment #8391240 -
Flags: approval-mozilla-aurora?
Updated•10 years ago
|
status-firefox29:
--- → affected
status-firefox30:
--- → fixed
Updated•10 years ago
|
Attachment #8391240 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment 14•10 years ago
|
||
Reproduced the initial issue on Surface Pro 2 (Windows 8.1 64bit) using Aurora from 2014-03-14, 125% and 200% DPI, verified as fixed using Firefox 29 RC and latest Aurora.
Status: RESOLVED → VERIFIED
Keywords: verifyme
You need to log in
before you can comment on or make changes to this bug.
Description
•