Closed
Bug 1045217
Opened 10 years ago
Closed 9 years ago
[10.10] Tooltips have an incorrect color: yellow, but should be grey
Categories
(Core :: Widget: Cocoa, defect)
Tracking
()
People
(Reporter: bugzilla, Assigned: Gijs)
References
(Blocks 1 open bug)
Details
(Whiteboard: [yosemite])
Attachments
(2 files, 1 obsolete file)
112.10 KB,
image/png
|
Details | |
1.36 KB,
patch
|
mstange
:
review+
lmandel
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
Build ID:Mozilla/5.0 (Macintosh; Intel Mac OS X 10.10; rv:34.0) Gecko/20100101 Firefox/34.0 STR: Hover any tab to see a tooltip with the website's full title/name Actual results: Tooltips are yellow Expected results: Tooltips should be grey like the rest of the tooltips in OS X 10.10
Reporter | ||
Updated•10 years ago
|
Summary: [10.10] Tooltips have the incorrect color: yellow, but should be grey → [10.10] Tooltips have an incorrect color: yellow, but should be grey
Assignee | ||
Updated•10 years ago
|
Component: General → Themes
Product: Firefox → Toolkit
Updated•10 years ago
|
Component: Themes → Widget: Cocoa
Product: Toolkit → Core
Assignee | ||
Comment 1•9 years ago
|
||
In my VM, the background is #f0f0f0, but I don't know whether the tooltips are using vibrancy/transparency, as the VM doesn't show any. In any case, the code that needs updating is here: http://mxr.mozilla.org/mozilla-central/source/widget/cocoa/nsNativeThemeCocoa.mm#2327 . We could also do with slightly more horizontal padding on the tooltips.
Assignee | ||
Comment 2•9 years ago
|
||
This should use the behind-window vibrancy on OS X. I expect the simplest will be to just update the -moz-appearance here: http://mxr.mozilla.org/mozilla-central/source/toolkit/themes/osx/global/popup.css#101 although I wonder if instead we should not change -moz-appearance: tooltip to use behind-window-vibrancy in cocoa instead... That seems cleaner and would also mean other apps/themes/things-with-moz-appearance-tooltip automatically benefit from the update. I'd be happy to look at this myself - Markus, could you give a pointer or two on how to do that, or is that trickier than it sounds? :-)
Assignee | ||
Comment 3•9 years ago
|
||
(In reply to :Gijs Kruitbosch from comment #2) > I'd be happy to look at this myself - > Markus, could you give a pointer or two on how to do that, or is that > trickier than it sounds? :-)
Flags: needinfo?(mstange)
Comment 4•9 years ago
|
||
FYI, we don't do more than just paint the yellow background-color for "-moz-appearance: tooltip;" ----------------------------------- 2374 case NS_THEME_TOOLTIP: 2375 CGContextSetRGBFillColor(cgContext, 0.996, 1.000, 0.792, 0.950); 2376 CGContextFillRect(cgContext, macRect); 2377 break;
Assignee | ||
Comment 5•9 years ago
|
||
(In reply to Stefan [:stefanh] from comment #4) > FYI, we don't do more than just paint the yellow background-color for > "-moz-appearance: tooltip;" > > ----------------------------------- > 2374 case NS_THEME_TOOLTIP: > 2375 CGContextSetRGBFillColor(cgContext, 0.996, 1.000, 0.792, 0.950); > 2376 CGContextFillRect(cgContext, macRect); > 2377 break; Yes, I know - I tried to point out as much in comment #1 . But as you can see from the screenshot, this is no longer "just" transparency, but there's some blurring involved as well. To get this right, we should use the behind-window vibrancy implemented in bug 1055614.
Comment 6•9 years ago
|
||
(In reply to :Gijs Kruitbosch from comment #2) > Markus, could you give a pointer or two on how to do that, or is that > trickier than it sounds? :-) It's not - should be rather simple, actually. In nsNativeThemeCocoa, don't paint anything for NS_THEME_TOOLTIP if VibrancyManager::SystemSupportsVibrancy(), and add NS_THEME_TOOLTIP handling near the places NS_THEME_MAC_VIBRANCY_LIGHT is used in nsDisplayList.cpp and nsChildView.mm.
Flags: needinfo?(mstange)
Assignee | ||
Comment 7•9 years ago
|
||
Per discussion on IRC, this is a start, but we need to (at a minimum) tell OS X that we want vibrancy even when the tooltip (window) doesn't have focus.
Attachment #8488605 -
Flags: feedback?(mstange)
Assignee | ||
Comment 8•9 years ago
|
||
Markus, do you have time to look at this "soon" ? If not, shall I put up a patch to at least adjust the color to a transparent grey so it doesn't look quite so harshly out of place, that we can uplift for 34? :-)
Flags: needinfo?(mstange)
Comment 9•9 years ago
|
||
Last time I looked at this I wasn't able to make tooltips use vibrancy, no matter what I tried. So yes, adjusting the color would be the right short-term fix.
Flags: needinfo?(mstange)
Assignee | ||
Comment 10•9 years ago
|
||
This looks right to me on my actual 10.10 machine in terms of transparency/shade of gray.
Attachment #8498171 -
Flags: review?(mstange)
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED
Comment 11•9 years ago
|
||
Comment on attachment 8498171 [details] [diff] [review] fix 10.10 tooltips to use grey instead of yellow, That's a very white-looking grey you have there.
Attachment #8498171 -
Flags: review?(mstange) → review+
Assignee | ||
Comment 12•9 years ago
|
||
Yeah, checked in with more grey-ish grey (but still pretty non-grey). remote: https://hg.mozilla.org/integration/fx-team/rev/124797ab4a5c Twiddling lots of flags for backlog things. Marco, can you add this? (bug 1063714 is currently blocked, and I worked on this (important for yosemite stuff) when I didn't know I was going to need to back out bug 1057166 - I'll get back to that tonight / tomorrow, as this is done now) I'll file a separate bug for making this use proper translucency.
Iteration: --- → 35.3
Points: --- → 1
Flags: qe-verify+
Flags: needinfo?(mmucci)
Flags: in-testsuite-
Flags: firefox-backlog+
https://hg.mozilla.org/mozilla-central/rev/124797ab4a5c
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla35
Updated•9 years ago
|
QA Contact: catalin.varga
Comment 15•9 years ago
|
||
Verified as fixed using the following environment: FF 35 Build Id: 20141002030202 Os: Mac Os X 10.10
Status: RESOLVED → VERIFIED
Assignee | ||
Comment 16•9 years ago
|
||
Comment on attachment 8498171 [details] [diff] [review] fix 10.10 tooltips to use grey instead of yellow, Approval Request Comment [Feature/regressing bug #]: yosemite [User impact if declined]: really bright yellow tooltips in a gone-grey-and-translucent OS [Describe test coverage new/current, TBPL]: nope [Risks and why]: pretty low; uses a different color somewhere in widget instead of the one it uses now [String/UUID change made/needed]: nope NB: please transplant the landed patch instead of the one on the bug.
Attachment #8498171 -
Flags: approval-mozilla-aurora?
Comment 17•9 years ago
|
||
Comment on attachment 8498171 [details] [diff] [review] fix 10.10 tooltips to use grey instead of yellow, Aurora+
Attachment #8498171 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment 18•9 years ago
|
||
[Tracking Requested - why for this release]:
status-firefox33:
--- → wontfix
status-firefox34:
--- → affected
status-firefox35:
--- → fixed
tracking-firefox34:
--- → +
Updated•9 years ago
|
Comment 20•9 years ago
|
||
Verified as fixed using: FF34 Build Id: 20141009004002 OS: Mac Os X 10.10
Updated•9 years ago
|
Attachment #8488605 -
Attachment is obsolete: true
Attachment #8488605 -
Flags: feedback?(mstange)
You need to log in
before you can comment on or make changes to this bug.
Description
•