Closed
Bug 305049
Opened 19 years ago
Closed 19 years ago
The EM ui tries to display the icons for disabled extensions
Categories
(Toolkit :: Add-ons Manager, defect)
Toolkit
Add-ons Manager
Tracking
()
RESOLVED
FIXED
People
(Reporter: robert.strong.bugs, Assigned: robert.strong.bugs)
References
Details
(Keywords: fixed1.8)
Attachments
(1 file)
|
1.65 KB,
patch
|
benjamin
:
review+
cbeard
:
approval1.8b4+
|
Details | Diff | Splinter Review |
This will fallback to using the default icon but it causes noises while doing
so. Patch coming up
|
Assignee | |
Comment 1•19 years ago
|
||
Attachment #193021 -
Flags: review?(benjamin)
|
||
Updated•19 years ago
|
Attachment #193021 -
Flags: review?(benjamin)
Attachment #193021 -
Flags: review+
Attachment #193021 -
Flags: approval1.8b4?
|
|
Assignee | |
Comment 2•19 years ago
|
||
Fixed on trunk - leaving open in case this gets 1.8b4 approvalnepo
|
|
Assignee | |
Comment 3•19 years ago
|
||
Value: lessens user confusion with js console messages. There were several
people that attributed the associated js console message was part of the recent
regression (see bug 304874).
Risk: very minimal. This code makes the generic extension icon display for
disabled extensions. Previously it would try to use the extension provided icon
and fallback to the generic icon if that failed for disabled extensions.
Testing: The patch is now on the trunk and I've used the patch for a couple of days.
|
||
Updated•19 years ago
|
Attachment #193021 -
Flags: approval1.8b4? → approval1.8b4+
|
|
Assignee | |
Comment 4•19 years ago
|
||
Fixed on branch and trunk.
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
|
|
||
Comment 5•19 years ago
|
||
*** Bug 305237 has been marked as a duplicate of this bug. ***
|
||
Updated•16 years ago
|
Product: Firefox → Toolkit
You need to log in
before you can comment on or make changes to this bug.
Description
•