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

VERIFIED FIXED

Status

()

defect
P1
normal
VERIFIED FIXED
4 years ago
4 years ago

People

(Reporter: Gijs, Unassigned)

Tracking

(Blocks 1 bug)

Trunk
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: fixed by bug 1173725)

Attachments

(4 attachments)

(Reporter)

Description

4 years ago
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

4 years ago
Flags: needinfo?
(Reporter)

Comment 2

4 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

4 years ago
(Reporter)

Comment 4

4 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

4 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

4 years ago
This is making me think it's actually an MS bug, though I'm not 100% sure.
(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.
(Reporter)

Updated

4 years ago
Duplicate of this bug: 1179352
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

4 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.
Checked out Windows 8.1, this isn't an issue there.

Comment 12

4 years ago
Still seems to be an issue with the latest build

Updated

4 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

4 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.
Works for me with the letest Nightly.
(Reporter)

Comment 15

4 years ago
Bug 1173725 should be fixing this.
Depends on: 1173725
(Reporter)

Updated

4 years ago
Duplicate of this bug: 1181552
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).
(Reporter)

Updated

4 years ago
Duplicate of this bug: 1182912
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

4 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

4 years ago
(I checked the registry, definitely running 10240)
(Reporter)

Comment 22

4 years ago
Fixed by bug 1173725, anyway.
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED

Comment 23

4 years ago
Definitely appears to be fixed on 41.0a2 (2015-07-16) on 10240.
Flags: needinfo?(dao)
(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
Duplicate of this bug: 1188953
Duplicate of this bug: 1184062
Whiteboard: fixed by bug 1173725
(Reporter)

Updated

4 years ago
Duplicate of this bug: 1191844

Updated

4 years ago
Duplicate of this bug: 1195074
You need to log in before you can comment on or make changes to this bug.