Clicking on the left side of the minimize button close the window on macOS
Categories
(Firefox :: Theme, defect, P3)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr60 | --- | unaffected |
firefox66 | --- | wontfix |
firefox67 | --- | wontfix |
firefox68 | --- | verified |
People
(Reporter: karlcow, Assigned: dao)
References
(Regression)
Details
(Keywords: regression)
Attachments
(2 files)
(deleted),
image/png
|
Details | |
(deleted),
text/x-phabricator-request
|
pascalc
:
approval-mozilla-beta-
|
Details |
Tested on Firefox Nightly 68.0a1 (2019-03-20) (64-bit)
with macOS 10.14.3 (18D109)
I can reproduce this report by a user on twitter.
https://twitter.com/AxelFavry/status/1108706032283516929
The cursor needs to be just beside the minimize button on the left side. There is a tiny range for it to happen but it's happening.
Reporter | ||
Updated•6 years ago
|
Updated•6 years ago
|
Comment 1•6 years ago
|
||
Times, can you find when this regressed?
Comment 2•6 years ago
|
||
Fwiw - I think this is more of a Widget :: Cocoa thing, from what I can tell from https://searchfox.org/mozilla-central/rev/56705678f5fc363be5e0237e1686f619b0d23009/widget/cocoa/nsNativeThemeCocoa.mm#392-413.
Updated•6 years ago
|
Comment 3•6 years ago
|
||
Huh, it looks like we have two sets of buttons there: The native window buttons on top, with correct hover feedback, and XUL toolbarbuttons underneath. The XUL toolbarbuttons are misplaced and have the wrong size.
If you click in the gaps between the native buttons, you hit the XUL buttons. And the XUL close button happens to sit right at the left edge of the native minimize button.
Comment 4•6 years ago
|
||
Oh, the XUL buttons also have the wrong order.
Can we just set the XUL buttons to display:none on Mac?
Comment 5•6 years ago
|
||
(In reply to Markus Stange [:mstange] from comment #4)
Oh, the XUL buttons also have the wrong order.
Can we just set the XUL buttons to display:none on Mac?
Yes, certainly - although that doesn't seem to help... if I manually remove those XUL nodes from the DOM, I'm still able to reproduce the issue.
Comment 6•6 years ago
|
||
Manually removing the nodes solves the problem for me. Did you remove the nodes in the right window / document?
Comment 7•6 years ago
|
||
Perhaps I didn't. I trust you on this, so let's move this to Theme to get these things display: none'd on macOS.
Comment 8•6 years ago
|
||
Here is the regression range:
Last good revision: 1ab00b6e99fd65b651381c235f1ae95ed746e57a
First bad revision: e354a018b703c453e8afcda1f44f1d51456536f5
Pushlog: https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=1ab00b6e99fd65b651381c235f1ae95ed746e57a&tochange=e354a018b703c453e8afcda1f44f1d51456536f5
Comment 9•6 years ago
|
||
Mike it sounds like this is from bug 1356920.
Updated•6 years ago
|
Updated•6 years ago
|
Assignee | ||
Updated•6 years ago
|
Assignee | ||
Comment 11•6 years ago
|
||
Assignee | ||
Updated•6 years ago
|
Comment 12•6 years ago
|
||
Comment on attachment 9063998 [details]
Bug 1538049 - Fix broken .titlebar-button selector. r=ntim
Beta/Release Uplift Approval Request
- User impact if declined: Glitch that's annoying to some users potentially causing data loss: https://twitter.com/AxelFavry/status/1108706032283516929
- Is this code covered by automated tests?: No
- Has the fix been verified in Nightly?: No
- Needs manual test from QE?: Yes
- If yes, steps to reproduce: See comment 0
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): one line CSS change fixing a broken selector
- String changes made/needed: no
Updated•6 years ago
|
Comment 13•6 years ago
|
||
Comment 14•6 years ago
|
||
bugherder |
Comment 15•6 years ago
|
||
Comment on attachment 9063998 [details]
Bug 1538049 - Fix broken .titlebar-button selector. r=ntim
This is coming a bit late in the cycle as we will build RC today. Given that this is a P3, mac only and in the DOM inspector, I don't think the impact justifies an uplift in RC week, sorry.
Comment 16•6 years ago
|
||
(In reply to Pascal Chevrel:pascalc from comment #15)
Comment on attachment 9063998 [details]
Bug 1538049 - Fix broken .titlebar-button selector. r=ntimThis is coming a bit late in the cycle as we will build RC today. Given that this is a P3, mac only and in the DOM inspector, I don't think the impact justifies an uplift in RC week, sorry.
This is not only in the DOM inspector, this is in the browser window itself, but I guess the change can wait until the next release.
Updated•6 years ago
|
Comment 17•6 years ago
|
||
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.13; rv:68.0) Gecko/20100101 Firefox/68.0
Build ID: 20190513082256
Verified as fixed on the latest Nightly build on macOS 10.13.
Assignee | ||
Updated•5 years ago
|
Description
•