Closed Bug 414564 Opened 17 years ago Closed 17 years ago

AMO Integration: Rating star icons need some love

Categories

(Firefox :: Theme, defect, P2)

x86
macOS
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: u88484, Assigned: shorlander)

References

Details

(Keywords: polish)

Attachments

(3 files)

Attached image Screenshot (deleted) —
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.
Blocks: 404024
Attached image Screenshot 2, shows other states (deleted) —
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...
(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.
Added to the icon inventory, will be getting new icons for all platforms.
Just waiting on the OSX icons to close this.
OS: Windows XP → Mac OS X
These look pretty bad, want to make sure this isn't lost.
Flags: blocking-firefox3?
>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.
Should this be in Firefox::Theme? I'll let Mossop decide, but I think so.
Flags: blocking-firefox3? → blocking-firefox3+
Priority: -- → P2
Component: Extension/Theme Manager → Theme
QA Contact: extension.manager → theme
Will these also match on amo or should that be spun off into another bug?
>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.
Assignee: nobody → stephen
Whiteboard: [needs status update]
We have the new icons from the IconFactory now, just a matter of slicing and dicing.
(In reply to comment #13) > We have the new icons from the IconFactory now, just a matter of slicing and > dicing. > Even for OSX?
Stephen created the new icons for OS X, not sure when they are landing.
(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.
The new icons for OSX have landed and look awesome
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Whiteboard: [needs status update]
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.

Attachment

General

Creator:
Created:
Updated:
Size: