Closed Bug 948568 Opened 7 years ago Closed 7 years ago

Use inverted chat toolbar icons with dark Personas

Categories

(Thunderbird :: Theme, defect)

All
Windows 7
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 29.0

People

(Reporter: Paenglab, Assigned: Paenglab)

Details

Attachments

(2 files)

The only toolbar not using inverted icons with dark Personas is the chat toolbar. This bug is to fix this.
The icons aren't new. I took them from addressbook-toolbar-aero-inverted.png. I only resized and doubled the Join Chat icon to have two bubbles like on the normal icon.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #8345434 - Flags: ui-review?(josiah)
Attachment #8345434 - Flags: review?(josiah)
Attached image chat.png
Screenshot for easier ui-r.
Comment on attachment 8345434 [details] [diff] [review]
chatInverted.patch

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

Yep, looks great. Thanks!

::: mail/themes/windows/mail/chat-aero.css
@@ +216,5 @@
> +toolbar:-moz-lwtheme-darktext #button-add-buddy[disabled],
> +toolbar:-moz-lwtheme-darktext #button-join-chat,
> +toolbar:-moz-lwtheme-darktext #button-join-chat[disabled],
> +toolbar:-moz-lwtheme-darktext #button-chat-accounts,
> +toolbar:-moz-lwtheme-darktext #button-chat-accounts[disabled] {

Note to self (or others): I really wish we had a class for all these buttons. Not in the scope of this bug though.
Attachment #8345434 - Flags: ui-review?(josiah)
Attachment #8345434 - Flags: ui-review+
Attachment #8345434 - Flags: review?(josiah)
Attachment #8345434 - Flags: review+
Keywords: checkin-needed
https://hg.mozilla.org/comm-central/rev/8c88c75fb2ff
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 29.0
You need to log in before you can comment on or make changes to this bug.