Closed Bug 421635 Opened 13 years ago Closed 13 years ago
[EM] post restart notification should specify foreground color
Bug 408115 introduced post-restart notifications. But |richlistitem[newAddon="true"]| doesn't have foreground color rule. They are hard to read on some OS themes. Let's specify foreground color.
Screenshot showing the problem.
Screenshot with patch.
Comment on attachment 308091 [details] [diff] [review] Patch v1 (winstripe and gnomestripe) Thanks for catching this. Could you also do the same for pinstripe?
Added pinstripe (|color| comes from |notification[type="warning"]|) http://mxr.mozilla.org/seamonkey/source/toolkit/themes/pinstripe/global/notification.css#14
Comment on attachment 308101 [details] [diff] [review] Patch v2 r=me - thank you much for this
Attachment #308101 - Flags: review?(robert.bugzilla) → review+
ccing Mossop so he knows about this change
Attachment #308101 - Flags: approval1.9?
Comment on attachment 308101 [details] [diff] [review] Patch v2 a1.9+=damons
Attachment #308101 - Flags: approval1.9? → approval1.9+
OS: Linux → All
Hardware: PC → All
Checking in toolkit/themes/gnomestripe/mozapps/extensions/extensions.css; /cvsroot/mozilla/toolkit/themes/gnomestripe/mozapps/extensions/extensions.css,v <-- extensions.css new revision: 1.16; previous revision: 1.15 done Checking in toolkit/themes/pinstripe/mozapps/extensions/extensions.css; /cvsroot/mozilla/toolkit/themes/pinstripe/mozapps/extensions/extensions.css,v <-- extensions.css new revision: 1.40; previous revision: 1.39 done Checking in toolkit/themes/winstripe/mozapps/extensions/extensions.css; /cvsroot/mozilla/toolkit/themes/winstripe/mozapps/extensions/extensions.css,v <-- extensions.css new revision: 1.49; previous revision: 1.48 done
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3 beta5
You need to log in before you can comment on or make changes to this bug.