Port bug 1475509 to TB: Stop setting custom text selection styles on dark/light themes

RESOLVED FIXED in Thunderbird 63.0

Status

defect
RESOLVED FIXED
9 months ago
9 months ago

People

(Reporter: Paenglab, Assigned: Paenglab)

Tracking

unspecified
Thunderbird 63.0

Thunderbird Tracking Flags

(thunderbird_esr60 fixed, thunderbird62 fixed, thunderbird63 fixed)

Details

Attachments

(2 attachments)

(Assignee)

Description

9 months ago
FX removed this because it doesn't fit with the dark theme on macOS 10.14.
(Assignee)

Comment 1

9 months ago
Port of https://hg.mozilla.org/mozilla-central/rev/b3b2ac62bbe0
We have some more lines for the compose window.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #8992173 - Flags: review?(jorgk)

Comment 2

9 months ago
Comment on attachment 8992173 [details] [diff] [review]
no-selection-color.patch

Thanks. This appears to change the selection colour from blue to "light" in the search boxes and all the fields in the compose window when using the dark theme. That certainly looks better.
Attachment #8992173 - Flags: review?(jorgk) → review+

Comment 3

9 months ago
BTW, I see no difference using the "light" theme.
(Assignee)

Comment 4

9 months ago
This is the patch for ESR60.

Yes on Win10 the colours are almost the same on Light theme.
Attachment #8992177 - Flags: approval-comm-esr60?

Comment 5

9 months ago
Comment on attachment 8992177 [details] [diff] [review]
no-selection-color-ESR60.patch

OK, but then it needs to go to beta, too. Are M-C uplifting their bug?
Attachment #8992177 - Flags: approval-comm-esr60?
Attachment #8992177 - Flags: approval-comm-esr60+
Attachment #8992177 - Flags: approval-comm-beta+
(Assignee)

Comment 6

9 months ago
They don't have it in ESR, Maybe they uplift it to 62 because macOS 10.14 will be released during 62. We then have ESR60 and then it should be removed too.

Comment 7

9 months ago
Well, in the Gloda text box, when using a dark theme, the selection colour is blue now in TB 60 ESR, so that's bad. So why wait if we can fix it before release? BTW, with the dark theme in TB 60 ESR, Gloda and QFB text boxes look differently. Gloda is dark and QFB light. That's a bug, no? Do you want a screenshot?

Comment 8

9 months ago
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/0bfa777345f9
Port bug 1475509 to TB: Stop setting custom text selection styles on dark/light themes. r=jorgk
Status: ASSIGNED → RESOLVED
Last Resolved: 9 months ago
Resolution: --- → FIXED

Updated

9 months ago
Target Milestone: --- → Thunderbird 63.0
(Assignee)

Comment 9

9 months ago
I see it too on ESR60. I'll fix it and file a bug.
You need to log in before you can comment on or make changes to this bug.