Tooltips now have yellow background instead of white
Categories
(Toolkit :: Themes, defect, P1)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr91 | --- | unaffected |
firefox99 | --- | unaffected |
firefox100 | --- | unaffected |
firefox101 | + | verified |
firefox102 | --- | verified |
People
(Reporter: saschanaz, Assigned: emilio)
References
(Regression)
Details
(Keywords: regression)
Attachments
(5 files)
7.23 KB,
image/png
|
Details | |
48 bytes,
text/x-phabricator-request
|
RyanVM
:
approval-mozilla-beta+
|
Details | Review |
48 bytes,
text/x-phabricator-request
|
RyanVM
:
approval-mozilla-beta+
|
Details | Review |
47.03 KB,
image/jpeg
|
Details | |
58.01 KB,
image/jpeg
|
Details |
I guess this is not an intended change per the description in https://phabricator.services.mozilla.com/D143754 ?
On regular (non-HCM) Windows 10+ we just fill a rect with infobackground color
and so on, so there shouldn't be any effective behavior change.
It did introduce a change, so filing a bug. Maybe a chance to introduce dark mode to tooltips 😄
Comment 1•1 year ago
|
||
Set release status flags based on info from the regressing bug 1757797
Comment 2•1 year ago
|
||
:emilio, since you are the author of the regressor, bug 1757797, could you take a look?
For more information, please visit auto_nag documentation.
Updated•1 year ago
|
Assignee | ||
Comment 3•1 year ago
|
||
That matches the native tooltips that already existed in the restore buttons and other applications, so even tho it was not directly intentional, it's it's pretty much a progression.
In terms of introducing dark mode support to tooltips, there's already a proton tooltips pref that does that's but it's not enabled by default. We could also add non-native theme support for the tooltip theme colors, but that would be inconsistent with the restore window tooltips and so.
Dao do you know the state of the proton tooltip pref? Presumably nobody is working on it? Maybe it could be cleaned up and we could improve a bit all the tooltips instead.
(In reply to Emilio Cobos Álvarez (:emilio) from comment #3)
That matches the native tooltips that already existed in the restore buttons and other applications ...
Only the window control buttons are yellow natively on Windows (on other apps and for native apps like File Explorer), any other tooltips are white afaict.
Updated•1 year ago
|
Updated•1 year ago
|
Comment 5•1 year ago
|
||
(In reply to Emilio Cobos Álvarez (:emilio) from comment #3)
Dao do you know the state of the proton tooltip pref? Presumably nobody is working on it? Maybe it could be cleaned up and we could improve a bit all the tooltips instead.
I haven't heard of any plans to do that and don't know what it would take, so I would rather not count on that to fix this bug.
Can we just switch to background-color: window
and color: windowtext
on Windows? Perhaps only for @media (-moz-windows-default-theme)
if we still want the info color for high contrast?
Updated•1 year ago
|
Assignee | ||
Comment 6•1 year ago
|
||
Updated•1 year ago
|
Assignee | ||
Comment 7•1 year ago
|
||
Sure, that works. I'll send another patch which looks slightly nicer as well.
Assignee | ||
Comment 8•1 year ago
|
||
For that, make the Windows rounded-rect code apply to tooltips as well.
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/b9b71e18f873 Use Window/WindowText for tooltips in the default Windows theme. r=dao https://hg.mozilla.org/integration/autoland/rev/34a84ab2978a Make tooltips look nicer with default window themes. r=dao
Comment 10•1 year ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/b9b71e18f873
https://hg.mozilla.org/mozilla-central/rev/34a84ab2978a
Updated•1 year ago
|
Assignee | ||
Comment 13•1 year ago
|
||
Comment on attachment 9274813 [details]
Bug 1765423 - Make tooltips look nicer with default window themes. r=dao
Beta/Release Uplift Approval Request
- User impact if declined: Tooltips look ugly-ish on Win10/11.
- 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: comment 0
- List of other uplifts needed: Bug 1767815
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): Simple CSS change.
- String changes made/needed: none
- Is Android affected?: No
Assignee | ||
Updated•1 year ago
|
Updated•1 year ago
|
Comment 14•1 year ago
|
||
Comment on attachment 9274807 [details]
Bug 1765423 - Use Window/WindowText for tooltips in the default Windows theme. r=dao
Approved for 101.0b5.
Updated•1 year ago
|
Comment 15•1 year ago
|
||
bugherder uplift |
Comment 16•1 year ago
|
||
Reproduced this issue on an affected Nightly build (2022-04-19) on Windows 10 x64.
Verified as fixed on Firefox 101.0b5 (20220510144626) and Firefox 102.0a1 (20220510213954) on Windows 10 x64 and Windows 11.
Comment 17•1 year ago
|
||
Tooltips seem to be less defined and harder to read with this change.
Where they previously had a light background and black border; they now have a darker background and grey border.
100:
101.0b5:
Comment 18•1 year ago
|
||
Comment 19•1 year ago
|
||
Updated•1 year ago
|
Updated•1 year ago
|
Updated•1 year ago
|
Description
•