Closed
Bug 1179283
Opened 9 years ago
Closed 9 years ago
On non-100% DPI windows 10, the cut-out for the titlebar buttons and/or its placeholder are too small, causing caption buttons to be cut off or overlap
Categories
(Core :: Widget: Win32, defect, P1)
Core
Widget: Win32
Tracking
()
VERIFIED
FIXED
People
(Reporter: Gijs, Unassigned)
References
Details
(Whiteboard: fixed by bug 1173725)
Attachments
(4 files)
This doesn't happen if my VM uses 100% DPI. If I change it to 150% it goes sadfaces. On things where we don't seem to cut out titlebar things (default theme) this is causing the minimize button to overlap with the tablist dropdown button (all-tabs button). On lightweight themes, we seem to be sizing the cutout for the titlebar buttons wrongly and so the minimize button is cut off instead. The height of the cutout is also wrong.
I don't see any of this on windows 8. I wonder if this is related to bug 1173732 and if this both are win10-only problems.
Dão, can you clarify if that bug happens on win8 or previous at all? I'm not 100% up-to-date on it (and don't have hidpi win8).
Flags: needinfo?
Reporter | ||
Comment 1•9 years ago
|
||
Flags: needinfo?
Reporter | ||
Comment 2•9 years ago
|
||
(In reply to :Gijs Kruitbosch from comment #0)
> bug 1173732 and if this both are win10-only problems.
>
> Dão, can you clarify if that bug happens on win8 or previous at all? I'm not
> 100% up-to-date on it (and don't have hidpi win8).
Erm, I don't know what happened with the needinfo here, but let's try again.
Flags: needinfo?(dao)
Reporter | ||
Comment 3•9 years ago
|
||
Reporter | ||
Comment 4•9 years ago
|
||
FWIW, this didn't happen pre-build-10158, so in theory maybe MS will fix it before release? Or, alternatively, it might be an intentional change.
Reporter | ||
Comment 5•9 years ago
|
||
Hm, this also results in the minimize button being rendered outside the window frame if the window is too small... o.O
Reporter | ||
Comment 6•9 years ago
|
||
This is making me think it's actually an MS bug, though I'm not 100% sure.
Comment 7•9 years ago
|
||
(In reply to :Gijs Kruitbosch from comment #6)
> Created attachment 8628315 [details]
> Minimize button outside window frame
>
> This is making me think it's actually an MS bug, though I'm not 100% sure.
I agree.
Comment 9•9 years ago
|
||
I can't check Windows 8 right now, but surely we'd have noticed by now if it was an issue there.
Flags: needinfo?(dao)
Reporter | ||
Comment 10•9 years ago
|
||
(In reply to Dão Gottwald [:dao] from comment #9)
> I can't check Windows 8 right now, but surely we'd have noticed by now if it
> was an issue there.
I don't actually understand bug 1173732, so, I don't know.
Comment 11•9 years ago
|
||
Checked out Windows 8.1, this isn't an issue there.
Comment 12•9 years ago
|
||
Still seems to be an issue with the latest build
Updated•9 years ago
|
Attachment #8630462 -
Attachment description: Win 10 10162 + Dev Edition 41.0a2 (2015-07-06) x64 e10s enable → Win 10 10162 + Dev Edition 41.0a2 (2015-07-06) x64 e10s enabled
Comment 13•9 years ago
|
||
Further,
Setting "layout.css.devPixelsPerPx" to 1 or 1.5 in about:config scales the tabs / etc, but does not solve the issue.
Checking "Disable display scaling on high DPI settings" in compatibility for the shortcut has no effect at all.
Updated•9 years ago
|
Blocks: windows-10-issues
Priority: -- → P1
Comment 14•9 years ago
|
||
Works for me with the letest Nightly.
Comment 17•9 years ago
|
||
This seems to happen in private browsing as well, where the PB label from the title bar overlaps the minimize button. See: http://i.imgur.com/61Og7s5.png - screenshot from a Microsoft Surface Pro 2 using Windows 10 Pro x64 (Insider Preview Build 10162) with Beta 40.0b2 (20150706172413).
Comment 19•9 years ago
|
||
It looks like this may have been fixed in build 10240? I didn't do a direct before-after comparison, though, and have only tested 150% so far.
Flags: needinfo?(gijskruitbosch+bugs)
Reporter | ||
Comment 20•9 years ago
|
||
(In reply to Dão Gottwald [:dao] from comment #19)
> It looks like this may have been fixed in build 10240? I didn't do a direct
> before-after comparison, though, and have only tested 150% so far.
I still saw this with lightweight themes in the build I just got (which I presume is 10240, but the build id on the bottom right of the desktop is gone, so I don't know for sure). Likewise, I still see overlap between the _/+\_ hover state and the minimize button when using the default theme (ie comparing the hover states). I'm also on 150%. Any idea what's different?
Flags: needinfo?(gijskruitbosch+bugs) → needinfo?(dao)
Reporter | ||
Comment 21•9 years ago
|
||
(I checked the registry, definitely running 10240)
Reporter | ||
Comment 22•9 years ago
|
||
Fixed by bug 1173725, anyway.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Comment 23•9 years ago
|
||
Definitely appears to be fixed on 41.0a2 (2015-07-16) on 10240.
Updated•9 years ago
|
Flags: needinfo?(dao)
Comment 24•9 years ago
|
||
(In reply to Scott Frost from comment #23)
> Definitely appears to be fixed on 41.0a2 (2015-07-16) on 10240.
Thanks!
Status: RESOLVED → VERIFIED
Updated•9 years ago
|
status-firefox42:
affected → ---
Whiteboard: fixed by bug 1173725
You need to log in
before you can comment on or make changes to this bug.
Description
•