Touch bar icons missing when only non-browser windows open
Categories
(Core :: Widget: Cocoa, defect, P2)
Tracking
()
People
(Reporter: sam, Assigned: bugzilla)
References
(Blocks 1 open bug)
Details
Attachments
(2 files)
(deleted),
image/png
|
Details | |
(deleted),
text/x-phabricator-request
|
pascalc
:
approval-mozilla-beta+
|
Details |
The icons in the macOS touch bar are missing (but text and buttons are still visible) if there is a Firefox window focused, but no browser window open.
STR:
- Launch Firefox on a touch bar-equipped machine.
- Close the browser window, then open either the About or Library windows.
- Observe touch bar buttons are visible but missing their icons (example screenshot attached).
Expected: Either the touch bar buttons themselves should not be present in this scenario, or they should be visible but with a disabled appearance.
System:
macOS 10.15.2
MacBook Pro (15-inch, 2018)
Nightly 74.0a1 (2020-01-09) (64-bit)
Comment 1•5 years ago
|
||
The priority flag is not set for this bug.
:spohl, could you have a look please?
For more information, please visit auto_nag documentation.
Updated•5 years ago
|
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 2•5 years ago
|
||
Comment 4•5 years ago
|
||
bugherder |
Comment 5•5 years ago
|
||
Is this something we should nominate for Beta uplift to ship in 74?
Assignee | ||
Comment 6•5 years ago
|
||
I'd first like to wait a few days to look out for regressions, although this wasn't a very significant patch so it should be fine. I'll leave ni? to remind myself to check back.
Assignee | ||
Comment 7•5 years ago
|
||
Comment on attachment 9125484 [details]
Bug 1608336 - Disable Touch Bar for non-chrome windows. r?mikedeboer
Beta/Release Uplift Approval Request
- User impact if declined: UI ugliness: the Touch Bar contains unlabelled and useless buttons under the conditions in comment 0.
- Is this code covered by automated tests?: No
- Has the fix been verified in Nightly?: Yes
- 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): Adds a null-check to disable Touch Bar behaviour in some contexts. No chance of spillover effects into non-Touch Bar components, and the Touch Bar is seen by a very small proportion of users.
- String changes made/needed:
Assignee | ||
Updated•5 years ago
|
Updated•5 years ago
|
Comment 8•5 years ago
|
||
Reproduced issue with the following setup:
macOS 10.15.3 / Nightly 74.0a1 (2020-01-08) (64-bit).
Fix verified with 75.0a1 (2020-02-16).
Comment 9•5 years ago
|
||
Comment on attachment 9125484 [details]
Bug 1608336 - Disable Touch Bar for non-chrome windows. r?mikedeboer
Low risk patch, verified on Nightly, uplift approved for 74?0b5, thanks.
Comment 10•5 years ago
|
||
bugherder uplift |
Comment 11•5 years ago
|
||
Fix verified with 74.0b5 on macOS 10.15.3.
"unlabelled and useless buttons" are not displayed.
Description
•