Checked QFB button should better work with Personas

RESOLVED FIXED in Thunderbird 13.0

Status

defect
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: Paenglab, Assigned: Paenglab)

Tracking

unspecified
Thunderbird 13.0
x86_64
macOS

Thunderbird Tracking Flags

(thunderbird11 fixed, thunderbird12 fixed)

Details

Attachments

(2 attachments)

Bug 728553 introduced a pressed state when checked. The button has then a fixed color which could make it bad readable on some Personas.
Posted patch PatchSplinter Review
Patch inheriting the color and text shadow. Also making the button slightly more transparent.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #600107 - Flags: ui-review?(bwinton)
Attachment #600107 - Flags: review?(bwinton)
Adding Mark because he saw this first.
Comment on attachment 600107 [details] [diff] [review]
Patch

Seems to look better, and the code is clean.  r=me, ui-r=me.

Thanks,
Blake.
Attachment #600107 - Flags: ui-review?(bwinton)
Attachment #600107 - Flags: ui-review+
Attachment #600107 - Flags: review?(bwinton)
Attachment #600107 - Flags: review+
Keywords: checkin-needed
Comment on attachment 600107 [details] [diff] [review]
Patch

Associated Bug 728553 is also in beta. Then this makes sense to put this also there.
Attachment #600107 - Flags: approval-comm-beta?
Attachment #600107 - Flags: approval-comm-aurora?
Checked in: http://hg.mozilla.org/comm-central/rev/2210f715d6df
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 13.0
Attachment #600107 - Flags: approval-comm-beta?
Attachment #600107 - Flags: approval-comm-beta+
Attachment #600107 - Flags: approval-comm-aurora?
Attachment #600107 - Flags: approval-comm-aurora+
You need to log in before you can comment on or make changes to this bug.