Closed
Bug 414564
Opened 17 years ago
Closed 17 years ago
AMO Integration: Rating star icons need some love
Categories
(Firefox :: Theme, defect, P2)
Tracking
()
VERIFIED
FIXED
People
(Reporter: u88484, Assigned: shorlander)
References
Details
(Keywords: polish)
Attachments
(3 files)
In the new AMO integrated panel, the rating star icons need some love. When an extension/theme is selected, a white outline appears around the star and the stars also look disfigured. See screenshot.
Comment 2•17 years ago
|
||
The 'code' to use my ratings image:
.addonRating[rating]{
display:-moz-box;
width:68px;
height:12px;
list-style-image:url(chrome://mozapps/skin/extensions/ratings.png)}
.addonRating[rating="0"]{-moz-image-region:rect(0 68px 12px 0)}
.addonRating[rating="1"]{-moz-image-region:rect(12px 68px 24px 0)}
.addonRating[rating="2"]{-moz-image-region:rect(24px 68px 36px 0)}
.addonRating[rating="3"]{-moz-image-region:rect(36px 68px 48px 0)}
.addonRating[rating="4"]{-moz-image-region:rect(48px 68px 60px 0)}
.addonRating[rating="5"]{-moz-image-region:rect(60px 68px 72px 0)}
.addonRating[rating="6"]{-moz-image-region:rect(72px 68px 84px 0)}
.addonRating[rating="7"]{-moz-image-region:rect(84px 68px 96px 0)}
.addonRating[rating="8"]{-moz-image-region:rect(96px 68px 108px 0)}
.addonRating[rating="9"]{-moz-image-region:rect(108px 68px 120px 0)}
.addonRating[rating="10"]{-moz-image-region:rect(120px 68px 132px 0)}
This supports the full rating scale, whereas the current version rounds down to whole stars...
Comment 3•17 years ago
|
||
(In reply to comment #2)
> Created an attachment (id=300924) [details]
> This is what I made for my themes, it also supports the full range of ratings
> (0-10)
Looks great to me! Just to confirm, if you're offering these for us to use please request ui-review on the attachment from madhava@mozilla.com
Where are we even getting this rating from? I haven't seen any ratings or been able to rate an add-on on AMO in ages.
Comment 5•17 years ago
|
||
Added to the icon inventory, will be getting new icons for all platforms.
Comment 7•17 years ago
|
||
These look pretty bad, want to make sure this isn't lost.
Flags: blocking-firefox3?
Comment 8•17 years ago
|
||
>These look pretty bad, want to make sure this isn't lost.
yeah, not interacting well with the various selection colors. We will need to refresh them in the next icon drop.
Comment 9•17 years ago
|
||
Should this be in Firefox::Theme? I'll let Mossop decide, but I think so.
Flags: blocking-firefox3? → blocking-firefox3+
Priority: -- → P2
Updated•17 years ago
|
Component: Extension/Theme Manager → Theme
QA Contact: extension.manager → theme
Reporter | ||
Comment 10•17 years ago
|
||
Will these also match on amo or should that be spun off into another bug?
Comment 11•17 years ago
|
||
>Will these also match on amo or should that be spun off into another bug?
They will look different on each platform. We may want have amo adapt a variety of icons like the stars, or the puzzle piece based on the user agent.
Updated•17 years ago
|
Assignee: nobody → stephen
Updated•17 years ago
|
Whiteboard: [needs status update]
Comment 13•17 years ago
|
||
We have the new icons from the IconFactory now, just a matter of slicing and dicing.
Comment 14•17 years ago
|
||
(In reply to comment #13)
> We have the new icons from the IconFactory now, just a matter of slicing and
> dicing.
>
Even for OSX?
Comment 15•17 years ago
|
||
Stephen created the new icons for OS X, not sure when they are landing.
Assignee | ||
Comment 16•17 years ago
|
||
(In reply to comment #15)
> Stephen created the new icons for OS X, not sure when they are landing.
>
Kevin and I are working on getting some stuff in before the the freeze. Since the icons are already finished we just need to update the CSS. They should make it in.
Comment 17•17 years ago
|
||
The new icons for OSX have landed and look awesome
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Whiteboard: [needs status update]
Comment 18•17 years ago
|
||
Verified fix on Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9pre) Gecko/2008041404 Minefield/3.0pre.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•