The default bug view has changed. See this FAQ.

about:newaddon doesn't restrict size of the add-on's icon

RESOLVED FIXED in mozilla23

Status

()

Toolkit
Add-ons Manager
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: Unfocused, Assigned: sachin)

Tracking

Trunk
mozilla23
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 1 obsolete attachment)

Created attachment 715729 [details]
Screenshot

Seems about:newaddon doesn't restrict the size of the add-on's icon, and blindly assumes the image is the right size. So if the add-on supplies a huge image for it's icon, it is shown at that huge size. See the attached screenshot for an example.
(Assignee)

Comment 1

4 years ago
I would like to take this bug up.
So, what exactly is the right size for an addon's icon?
Flags: needinfo?(bmcbride)
(Assignee)

Comment 2

4 years ago
Created attachment 738606 [details] [diff] [review]
Sets the image's width to 64px and height to 'auto'

It doesn't check the present size however, since I don't know what is a right size.
Please tell me if this is what I should be doing.
Attachment #738606 - Flags: feedback?(bmcbride)
Comment on attachment 738606 [details] [diff] [review]
Sets the image's width to 64px and height to 'auto'

Review of attachment 738606 [details] [diff] [review]:
-----------------------------------------------------------------

We support 32x32 and 64x64 sized icons for this. The way other code handles this is to set max-width:64px; max-height:64px; in the CSS file - that way the image will automatically show as its original size (without scaling) if it's not too big, or be scaled down to 64x64 if it's too big.
Attachment #738606 - Flags: feedback?(bmcbride) → feedback-
Flags: needinfo?(bmcbride)
(Btw, thanks for picking up all these bugs. Sorry for any delays in replying - I've been off sick.)
Assignee: nobody → sachinhosmani2
Status: NEW → ASSIGNED
(Assignee)

Comment 5

4 years ago
Created attachment 740674 [details] [diff] [review]
Sets the max-height and max-weight CSS properties.
Attachment #738606 - Attachment is obsolete: true
Attachment #740674 - Flags: review?(bmcbride)
(Assignee)

Comment 6

4 years ago
(In reply to Blair McBride [:Unfocused] (Back from the dead. Mostly.) from comment #4)
> (Btw, thanks for picking up all these bugs. Sorry for any delays in replying
> - I've been off sick.)

No problem. Thank you :)
Attachment #740674 - Flags: review?(bmcbride) → review+
https://hg.mozilla.org/integration/fx-team/rev/6b6e79e65b37
Flags: in-testsuite-
https://hg.mozilla.org/mozilla-central/rev/6b6e79e65b37
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla23
You need to log in before you can comment on or make changes to this bug.