Last Comment Bug 747165 - Chat icons not shown on Aero in small icon mode
: Chat icons not shown on Aero in small icon mode
Product: Thunderbird
Classification: Client Software
Component: Instant Messaging (show other bugs)
: unspecified
: All Windows 7
: -- normal (vote)
: Thunderbird 14.0
Assigned To: Richard Marti (:Paenglab)
Depends on:
  Show dependency treegraph
Reported: 2012-04-19 13:55 PDT by Richard Marti (:Paenglab)
Modified: 2012-04-23 07:54 PDT (History)
3 users (show)
ryanvm: in‑testsuite-
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---

Fix (1.11 KB, patch)
2012-04-19 13:58 PDT, Richard Marti (:Paenglab)
mconley: review+
Details | Diff | Splinter Review

Description Richard Marti (:Paenglab) 2012-04-19 13:55:30 PDT
When the toolbar is in small icons mode the icons aren't shown because the XP selectors are stronger.
Comment 1 Richard Marti (:Paenglab) 2012-04-19 13:58:25 PDT
Created attachment 616738 [details] [diff] [review]

Adding !important solves this issue. I also fixed a copy n paste error which also made the disabled Add buddy icon disappear.

Mike, I hope it's okay to ask you for review. You're so a fast reviewer :)
Comment 2 Mike Conley (:mconley) 2012-04-20 13:46:30 PDT
Comment on attachment 616738 [details] [diff] [review]

Review of attachment 616738 [details] [diff] [review]:

Works as advertised, and code looks good.

Thanks Richard,

Comment 3 Richard Marti (:Paenglab) 2012-04-20 13:49:00 PDT
Mike, thank you for the fast review.
Comment 4 Mike Conley (:mconley) 2012-04-20 13:49:33 PDT

No problem!  Thanks for your great work!

Comment 5 Richard Marti (:Paenglab) 2012-04-20 13:50:36 PDT
Comment on attachment 616738 [details] [diff] [review]

[Approval Request Comment]
The same issue exists also on Aurora. To work correctly this should also land there.
Comment 6 Ryan VanderMeulen [:RyanVM] 2012-04-20 14:21:49 PDT
Comment 7 Florian Quèze [:florian] [:flo] 2012-04-23 07:54:07 PDT
Comment on attachment 616738 [details] [diff] [review]

Removing the approval-comm-aurora flag as I think we should rather take the patch from bug 735702 (which includes a fix for the same issue, see bug 735702 comment 21) to aurora if we still want to fix icon stuff there.

Note You need to log in before you can comment on or make changes to this bug.