AMO-provided icons aren't always 32x32

RESOLVED FIXED in mozilla1.9beta4

Status

()

Toolkit
Add-ons Manager
--
minor
RESOLVED FIXED
10 years ago
10 years ago

People

(Reporter: mossop, Assigned: mossop)

Tracking

Trunk
mozilla1.9beta4
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

10 years ago
Some icons for addons on amo aren't 32x32 so currently display stretched in the UI.
(Assignee)

Comment 1

10 years ago
Created attachment 301263 [details] [diff] [review]
patch rev 1

This makes the outer container of the icons 32x32 centering the icon within. Also fixes the same issue in the xpinstall confirmation dialog.
Assignee: nobody → dtownsend
Status: NEW → ASSIGNED
Attachment #301263 - Flags: review?(robert.bugzilla)
Attachment #301263 - Flags: review?(robert.bugzilla) → review+
(Assignee)

Comment 2

10 years ago
Comment on attachment 301263 [details] [diff] [review]
patch rev 1

Seeking approval to land post beta 3
Attachment #301263 - Flags: approval1.9?
(Assignee)

Updated

10 years ago
Whiteboard: [has patch]

Updated

10 years ago
Attachment #301263 - Flags: approval1.9? → approval1.9+
Summary: AMO provided icons aren't always 32x32 → AMO-provided icons aren't always 32x32
(Assignee)

Updated

10 years ago
Whiteboard: [has patch]
(Assignee)

Comment 3

10 years ago
Checking in toolkit/mozapps/extensions/content/extensions.xml;
/cvsroot/mozilla/toolkit/mozapps/extensions/content/extensions.xml,v  <--  extensions.xml
new revision: 1.55; previous revision: 1.54
done
Checking in toolkit/mozapps/xpinstall/content/xpinstallItem.xml;
/cvsroot/mozilla/toolkit/mozapps/xpinstall/content/xpinstallItem.xml,v  <--  xpinstallItem.xml
new revision: 1.3; previous revision: 1.2
done
Checking in toolkit/themes/pinstripe/mozapps/xpinstall/xpinstallConfirm.css;
/cvsroot/mozilla/toolkit/themes/pinstripe/mozapps/xpinstall/xpinstallConfirm.css,v  <--  xpinstallConfirm.css
new revision: 1.4; previous revision: 1.3
done
Checking in toolkit/themes/winstripe/mozapps/xpinstall/xpinstallConfirm.css;
/cvsroot/mozilla/toolkit/themes/winstripe/mozapps/xpinstall/xpinstallConfirm.css,v  <--  xpinstallConfirm.css
new revision: 1.8; previous revision: 1.7
done
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
(Assignee)

Updated

10 years ago
Target Milestone: --- → Firefox 3 beta4
Product: Firefox → Toolkit
You need to log in before you can comment on or make changes to this bug.