Closed
Bug 423137
Opened 16 years ago
Closed 16 years ago
Handle ratings change from bug 417442
Categories
(Toolkit :: Add-ons Manager, defect, P1)
Toolkit
Add-ons Manager
Tracking
()
VERIFIED
FIXED
mozilla1.9beta5
People
(Reporter: mossop, Assigned: mossop)
References
Details
(Keywords: relnote)
Attachments
(2 files, 1 obsolete file)
874 bytes,
patch
|
robert.strong.bugs
:
review+
|
Details | Diff | Splinter Review |
3.69 KB,
patch
|
robert.strong.bugs
:
review+
|
Details | Diff | Splinter Review |
AMO are planning on dropping their maximum rating from 10 to 5. We need to handle that. Most likely the easiest is just to double the value in nsAddonRepository.js before passing out. The alternative involves changing the idl contract and the skin code as well.
Flags: blocking-firefox3?
Assignee | ||
Comment 1•16 years ago
|
||
This is the trivial version of the fix. Means the nsIAddonRepository still outputs ratings from 0-10
Attachment #309644 -
Flags: review?(robert.bugzilla)
Assignee | ||
Updated•16 years ago
|
Status: NEW → ASSIGNED
Whiteboard: [has patch]
Comment 2•16 years ago
|
||
Comment on attachment 309644 [details] [diff] [review] trivial patch r=me Is changing the idl contract and the skin code a better approach? Seems better for the long term.
Attachment #309644 -
Flags: review?(robert.bugzilla) → review+
Assignee | ||
Comment 3•16 years ago
|
||
Maybe it is. This is the fuller patch, adjusts the idl comment and changes the style rules to deal with 0-5 values. Wasn't sure if we're expected to rev the uuid when results from interfaces change like that so I just did anyway.
Attachment #310063 -
Flags: review?(robert.bugzilla)
Updated•16 years ago
|
Flags: blocking-firefox3? → blocking-firefox3+
Priority: -- → P1
Comment 4•16 years ago
|
||
Comment on attachment 310063 [details] [diff] [review] fuller patch surely that isn't the patch?
Attachment #310063 -
Flags: review?(robert.bugzilla) → review-
Assignee | ||
Comment 5•16 years ago
|
||
Ummm oops!
Attachment #310063 -
Attachment is obsolete: true
Attachment #310068 -
Flags: review?(robert.bugzilla)
Comment 6•16 years ago
|
||
Comment on attachment 310068 [details] [diff] [review] the real patch r=me the uuid change isn't necessary.
Attachment #310068 -
Flags: review?(robert.bugzilla) → review+
Updated•16 years ago
|
Whiteboard: [has patch] → [has patch][has reviews]
Assignee | ||
Comment 7•16 years ago
|
||
Fixed in the code. This will look a little odd till the server side is updated which is now scheduled for Thursday. Checking in toolkit/mozapps/extensions/public/nsIAddonRepository.idl; /cvsroot/mozilla/toolkit/mozapps/extensions/public/nsIAddonRepository.idl,v <-- nsIAddonRepository.idl new revision: 1.2; previous revision: 1.1 done Checking in toolkit/themes/gnomestripe/mozapps/extensions/extensions.css; /cvsroot/mozilla/toolkit/themes/gnomestripe/mozapps/extensions/extensions.css,v <-- extensions.css new revision: 1.18; previous revision: 1.17 done Checking in toolkit/themes/pinstripe/mozapps/extensions/extensions.css; /cvsroot/mozilla/toolkit/themes/pinstripe/mozapps/extensions/extensions.css,v <-- extensions.css new revision: 1.41; previous revision: 1.40 done Checking in toolkit/themes/winstripe/mozapps/extensions/extensions.css; /cvsroot/mozilla/toolkit/themes/winstripe/mozapps/extensions/extensions.css,v <-- extensions.css new revision: 1.50; previous revision: 1.49 done
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Whiteboard: [has patch][has reviews]
Target Milestone: --- → Firefox 3 beta5
star image is broken ? http://img164.imageshack.us/img164/2607/a3iw2.jpg
Comment 9•16 years ago
|
||
I have a theme based on the official code, and I solved this problem adding a default -moz-image-region attribute to .addonRating[rating], and marking the specific ones as !important. Hope this helps.
Comment 10•16 years ago
|
||
(In reply to comment #8) > star image is broken ? > > http://img164.imageshack.us/img164/2607/a3iw2.jpg > (In reply to comment #9) > I have a theme based on the official code, and I solved this problem adding a > default -moz-image-region attribute to .addonRating[rating], and marking the > specific ones as !important. Hope this helps. > Forgot to say it was replying to comment #8.
Assignee | ||
Comment 11•16 years ago
|
||
(In reply to comment #8) > star image is broken ? > > http://img164.imageshack.us/img164/2607/a3iw2.jpg Please read comment 7 where I pointed out that this will look a little odd until the server side is fixed tomorrow.
Comment 12•16 years ago
|
||
Verified fix on Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9pre) Gecko/2008050206 Minefield/3.0pre
Status: RESOLVED → VERIFIED
Updated•16 years ago
|
Product: Firefox → Toolkit
You need to log in
before you can comment on or make changes to this bug.
Description
•