Selected text background is the wrong color when window is not focused

RESOLVED FIXED in Firefox 53

Status

()

RESOLVED FIXED
9 years ago
2 years ago

People

(Reporter: jrmuizel, Assigned: stefanh)

Tracking

Trunk
mozilla53
x86
Mac OS X
Points:
---

Firefox Tracking Flags

(firefox53 fixed)

Details

Attachments

(1 attachment)

(Reporter)

Description

9 years ago
In text areas selected text has a white background when it should be light grey. In regular content selected text has a dark grey background when it should be light grey.
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → DUPLICATE
Duplicate of bug: 141007
Hmm, ok, the second issue is actually not covered by bug 141007.
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
(Assignee)

Comment 3

2 years ago
So we're not using a system color here, instead we're using the same (hardcoded) color for all platforms: https://dxr.mozilla.org/mozilla-central/rev/5e76768327660437bf3486554ad318e4b70276e1/widget/nsXPLookAndFeel.cpp#775

Looking at webkit, it seems they're using the NSColor secondarySelectedControlColor which is lighter:
https://github.com/WebKit/webkit/blob/1f6436eabdb0d90e04bad8af2c9ba50c6b6c8f98/Source/WebCore/rendering/RenderThemeMac.mm#L301
Assignee: nobody → stefanh
Status: REOPENED → NEW
(Assignee)

Comment 4

2 years ago
Created attachment 8810987 [details] [diff] [review]
Use MacOS system color for selected text background in inactive windows

Not sure if it's better to move everything to each platform-specific nsLookAndFeel file instead of the #ifndef... This is a bit more economical, though.
Attachment #8810987 - Flags: review?(mstange)
(Assignee)

Updated

2 years ago
Status: NEW → ASSIGNED
Comment on attachment 8810987 [details] [diff] [review]
Use MacOS system color for selected text background in inactive windows

Review of attachment 8810987 [details] [diff] [review]:
-----------------------------------------------------------------

Sorry for the delay, this looks good.
Attachment #8810987 - Flags: review?(mstange) → review+

Comment 6

2 years ago
Pushed by stefanh@inbox.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/befc69f2cf27
Use MacOS system color for selected text background in inactive windows. r=mstange.

Comment 7

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/befc69f2cf27
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago2 years ago
status-firefox53: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
QA Whiteboard: [good first verify]
You need to log in before you can comment on or make changes to this bug.