Closed
Bug 425116
Opened 17 years ago
Closed 17 years ago
Notification bars in addons manager use the wrong icon
Categories
(Toolkit :: Add-ons Manager, defect)
Toolkit
Add-ons Manager
Tracking
()
VERIFIED
FIXED
mozilla1.9
People
(Reporter: faaborg, Assigned: dao)
References
Details
(Keywords: polish)
Attachments
(2 files)
(deleted),
image/png
|
Details | |
(deleted),
patch
|
mossop
:
review+
mtschrep
:
approval1.9+
|
Details | Diff | Splinter Review |
Notification bars in the addons manager are currently using:
http://mxr.mozilla.org/seamonkey/source/toolkit/themes/winstripe/mozapps/extensions/question.png
However, these notification bars do not actually ask the user a question. Once an information icon is landed in toolkit, these notification bars should use it instead.
Reporter | ||
Comment 2•17 years ago
|
||
yep http://mxr.mozilla.org/seamonkey/source/toolkit/themes/winstripe/global/icons/information-16.png
Blocks: 428923
Reporter | ||
Comment 3•17 years ago
|
||
wait, actually we should take care of this problem with the new notification bar styling, where for information we use:
background http://mxr.mozilla.org/seamonkey/source/toolkit/themes/winstripe/global/icons/informationBar-60.png
icon http://mxr.mozilla.org/seamonkey/source/toolkit/themes/winstripe/global/icons/informationBarIcon-16.png
I'm going to dupe this to the bug for making those changes.
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → DUPLICATE
Updated•17 years ago
|
Flags: blocking-firefox3?
Assignee | ||
Comment 4•17 years ago
|
||
This is basically a bug in extensions.js.
Blocks: 348576
Status: RESOLVED → REOPENED
Component: Theme → Extension/Theme Manager
Resolution: DUPLICATE → ---
Assignee | ||
Updated•17 years ago
|
Assignee | ||
Comment 5•17 years ago
|
||
Robert: question.png was introduced in bug 329045 for notifications that don't ask any question. Please comment if I'm missing something.
Blocks: 329045
Comment 6•17 years ago
|
||
It was but bug 329045 does not depend on this bug being fixed for it to be fixed... especially since that bug was specifically for Firefox 2.0.
btw: some of the notifications do ask a question so both the info and the question icons will be needed.
Assignee | ||
Comment 7•17 years ago
|
||
Yes, we should take a close look at the current notifications and use info/warning/question icons as appropriate.
I added the dependency merely for the curious since the parts that are relevant for this bug made it into 1.9.
Assignee | ||
Comment 8•17 years ago
|
||
We really don't seem to ask a question in any of the notifications ... once we do, we should use chrome://global/skin/icons/question-16.png, which allows us to remove question.png.
Assignee: nobody → dao
Status: REOPENED → ASSIGNED
Attachment #320158 -
Flags: review?(robert.bugzilla)
Comment 9•17 years ago
|
||
Comment on attachment 320158 [details] [diff] [review]
patch
Looks ok to me
Attachment #320158 -
Flags: review?(robert.bugzilla) → review+
Assignee | ||
Updated•17 years ago
|
Attachment #320158 -
Flags: approval1.9?
Comment 10•17 years ago
|
||
Comment on attachment 320158 [details] [diff] [review]
patch
a+ schrep - need you guys to test the nighties for this tomorrow since we don't have any room for respins.
Attachment #320158 -
Flags: approval1.9? → approval1.9+
Assignee | ||
Updated•17 years ago
|
Keywords: checkin-needed
Comment 11•17 years ago
|
||
Checking in toolkit/mozapps/extensions/content/extensions.js;
/cvsroot/mozilla/toolkit/mozapps/extensions/content/extensions.js,v <-- extensions.js
new revision: 1.177; previous revision: 1.176
done
Checking in toolkit/themes/pinstripe/mozapps/jar.mn;
/cvsroot/mozilla/toolkit/themes/pinstripe/mozapps/jar.mn,v <-- jar.mn
new revision: 1.29; previous revision: 1.28
done
Checking in toolkit/themes/winstripe/mozapps/jar.mn;
/cvsroot/mozilla/toolkit/themes/winstripe/mozapps/jar.mn,v <-- jar.mn
new revision: 1.38; previous revision: 1.37
done
Status: ASSIGNED → RESOLVED
Closed: 17 years ago → 17 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3
Updated•17 years ago
|
Keywords: checkin-needed
Comment 12•17 years ago
|
||
The notification icon is shown with Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9pre) Gecko/2008050920 Minefield/3.0pre => VERIFIED.
Aleksej, how does it look like on Linux? Is the selected GTK2 theme's default notification icon shown?
OS: Windows XP → All
Hardware: PC → All
Comment 13•17 years ago
|
||
Sorry, my fault. This is XP only => Verified.
Status: RESOLVED → VERIFIED
OS: All → Windows XP
Hardware: All → PC
Updated•16 years ago
|
Product: Firefox → Toolkit
You need to log in
before you can comment on or make changes to this bug.
Description
•