Closed Bug 1457320 Opened 7 years ago Closed 7 years ago

Port bug 1445671 to TB: Stop using -moz-font-smoothing-background-color for the selected tab when using a lightweight theme

Categories

(Thunderbird :: Theme, enhancement)

All
macOS
enhancement
Not set
normal

Tracking

(thunderbird60 fixed, thunderbird61 fixed)

RESOLVED FIXED
Thunderbird 61.0
Tracking Status
thunderbird60 --- fixed
thunderbird61 --- fixed

People

(Reporter: Paenglab, Assigned: Paenglab)

Details

Attachments

(1 file)

In bug 14456671 is written that it's only visible on some LW-themes. We should follow to be on the safe side.
Direct port of https://hg.mozilla.org/mozilla-central/rev/7b8e832071f1 The only difference is that we use [selected="true"] instead of [visuallyselected=true].
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #8971422 - Flags: review?(jorgk)
Comment on attachment 8971422 [details] [diff] [review] font-smoothing.patch (In reply to Richard Marti (:Paenglab) from comment #1) > The only difference is that we use [selected="true"] instead of > [visuallyselected=true]. Can you explain that difference?
Attachment #8971422 - Flags: review?(jorgk) → review+
I'm not 100% sure, but I think visuallyselected is only set when the tab is selected and visible on the tab bar and not in the overflow. Selected is also set when it's in the overflow and not visible but selected.
Keywords: checkin-needed
Pushed by mozilla@jorgk.com: https://hg.mozilla.org/comm-central/rev/2ca60f1a2b5c Port bug 1445671 to TB: Stop using -moz-font-smoothing-background-color for the selected tab when using a lightweight theme. r=jorgk
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 61.0
Comment on attachment 8971422 [details] [diff] [review] font-smoothing.patch It's for an Edge case but then we are on the safe side. And later uplifts to this file apply better. Should be checked-in after bug 1457712 to not rot its beta patch.
Attachment #8971422 - Flags: approval-comm-beta?
Attachment #8971422 - Flags: approval-comm-beta? → approval-comm-beta+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: