Closed Bug 633847 Opened 11 years ago Closed 11 years ago

White-on-yellow close button is hard to see (missing-plugin infobar)

Categories

(Toolkit :: Themes, defect)

x86
macOS
defect
Not set
trivial

Tracking

()

RESOLVED FIXED
mozilla2.0b12

People

(Reporter: jruderman, Assigned: rik)

References

Details

(Keywords: regression)

Attachments

(5 files)

Mozilla/5.0 (Macintosh; Intel Mac OS X 10.5; rv:2.0b12pre) Gecko/20110212 Firefox/4.0b12pre

I think this is a regression from bug 589219, which added toolkit/themes/pinstripe/global/notification/close.png.
Yeah, that's kinda ugly.
True.

Do you have a page to reproduce that? I can't find a page that triggers this yellow notification.
Tried using the standard icon for warning messages.

+notification[type="warning"] .messageCloseButton {
+  list-style-image: url("chrome://global/skin/icons/close.png");
+}
Is it ok to reuse an image in another folder?
Oh and I should add that there is also a red version of this bar. [type=critical]
Attached patch PatchSplinter Review
Assignee: nobody → anthony
Status: NEW → ASSIGNED
Attachment #513071 - Flags: review?(dao)
Attachment #513071 - Flags: review?(dao) → review+
Component: Theme → Themes
Product: Firefox → Toolkit
QA Contact: theme → themes
Severity: minor → trivial
These yellow and red bars do not exist in other platforms?
This is the same patch with additional mercurial infos for checkin.
Keywords: checkin-needed
Attachment #513071 - Flags: approval2.0?
BTW, forget comment 7... I should think of using my brain sometimes...
Comment on attachment 513071 [details] [diff] [review]
Patch

a=beltzner
Attachment #513071 - Flags: approval2.0? → approval2.0+
Pushed:
http://hg.mozilla.org/mozilla-central/rev/3a1bdbbca259
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b12
You need to log in before you can comment on or make changes to this bug.