Closed Bug 1772018 Opened 2 years ago Closed 2 years ago

Tooltips are difficult to see

Categories

(Firefox :: Theme, defect, P2)

Firefox 101
Desktop
Windows 10
defect

Tracking

()

VERIFIED FIXED
103 Branch
Tracking Status
firefox-esr91 --- unaffected
firefox-esr102 102+ verified
firefox101 --- wontfix
firefox102 --- verified
firefox103 --- verified

People

(Reporter: nico.rieck, Assigned: emilio)

References

(Regression)

Details

(Keywords: regression)

Attachments

(1 file)

Since Version 101 tooltips on Windows are difficult to see. Not only do they look alien compared to OS tooltips, but they have worse contrast and don't stand out. They also lack shadows, which isn't a new bug, but makes the result even worse.

Could you please provide more details maybe with a screenshot to make it more clear for me to understand the issue?
Thanks.

Flags: needinfo?(nico.rieck)

Found another comment for the same problem. See the screenshots here for a comparison: https://bugzilla.mozilla.org/show_bug.cgi?id=1765423#c18 and https://bugzilla.mozilla.org/show_bug.cgi?id=1765423#c19

Flags: needinfo?(nico.rieck)

Emilio, how about we make the border more pronounced and the text fully black? Was there some particular reason for tuning them down?

Severity: -- → S3
Flags: needinfo?(emilio)
Keywords: regression
Priority: -- → P2
Regressed by: 1765423

I didn't tune them down, I just used the menu colors, but sure happy to adjust.

I guess menus are bigger so they are more distinguishable... Do you have any particular colors I should crib from? Generally I prefer not making up colors :)

Flags: needinfo?(emilio) → needinfo?(dao+bmo)

Set release status flags based on info from the regressing bug 1765423

(In reply to Emilio Cobos Álvarez (:emilio) from comment #4)

I didn't tune them down, I just used the menu colors,

Ah, I forgot about that and apparently even missed the comment in tooltip.css...

I guess menus are bigger so they are more distinguishable...

Yeah, I think that's fair.

Do you have any particular colors I should crib from? Generally I prefer not making up colors :)

I would recommend we make the text fully black or white, respectively. Perhaps we can just take that color to the border as well, as done in the fallback styling? If that looks too stark, my next try would be to go to the middle between #cfcfd8 and black, so something like #67676C. No need to overthink this, feel free to just make something up that feels better than the status quo, and we can always tweak it further later if needed.

Flags: needinfo?(dao+bmo)
Flags: needinfo?(emilio)
Has Regression Range: --- → yes

The bug has a release status flag that shows some version of Firefox is affected, thus it will be considered confirmed.

Status: UNCONFIRMED → NEW
Ever confirmed: true
Assignee: nobody → emilio
Status: NEW → ASSIGNED
Flags: needinfo?(emilio)
Pushed by ealvarez@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0e2af6833a15
Tweak tooltip colors on Windows. r=dao
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 103 Branch
Flags: qe-verify+

Emilio, do we want to uplift this color tweak in a 102 dot release and ESR102? Thanks

Flags: needinfo?(emilio)

I managed to reproduce this issue on Firefox 102.0(build ID: 20220623063721) on Windows 10 64-bits. Verified as fixed on Firefox 103.0b4(build ID: 20220703190044) and Nightly 104.0a1(build ID: 20220703213709) on Windows 10 64-bits.

Comment on attachment 9281513 [details]
Bug 1772018 - Tweak tooltip colors on Windows. r=dao

ESR Uplift Approval Request

  • If this is not a sec:{high,crit} bug, please state case for ESR consideration: Minor and simple color tweak to improve contrast.
  • User impact if declined:
  • Fix Landed on Version: 103
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): Simple CSS change.

Beta/Release Uplift Approval Request

  • User impact if declined: See above.
  • 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:
  • List of other uplifts needed: none
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): Simple CSS tweak.
  • String changes made/needed: none
  • Is Android affected?: No
Flags: needinfo?(emilio)
Attachment #9281513 - Flags: approval-mozilla-release?
Attachment #9281513 - Flags: approval-mozilla-esr102?

Comment on attachment 9281513 [details]
Bug 1772018 - Tweak tooltip colors on Windows. r=dao

Approved for 102.0.1, thanks.

Attachment #9281513 - Flags: approval-mozilla-release? → approval-mozilla-release+

Hello,
I confirm this issue is fixed on Firefox 102.0.1(build ID: 20220705093820) on Windows 10.

Status: RESOLVED → VERIFIED
Flags: qe-verify+

Comment on attachment 9281513 [details]
Bug 1772018 - Tweak tooltip colors on Windows. r=dao

Approved for 102.1esr.

Attachment #9281513 - Flags: approval-mozilla-esr102? → approval-mozilla-esr102+

Hello,
I confirm this fix is verified on Firefox 102.0.1esr(build ID: 20220711174428) on Windows 10.

You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: