Closed
Bug 424262
Opened 16 years ago
Closed 16 years ago
Ratings stars show a compressed composite of entire image block
Categories
(Toolkit :: Add-ons Manager, defect, P1)
Tracking
()
VERIFIED
FIXED
mozilla1.9beta5
People
(Reporter: ehume, Assigned: mossop)
Details
Attachments
(4 files, 2 obsolete files)
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9b5pre) Gecko/2008032005 Minefield/3.0b5pre Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9b5pre) Gecko/2008032005 Minefield/3.0b5pre The ratings icon shows the appropriate stars on the top recommended add-on, but shows a vertically-compressed composite of the entire image block in subsequent recommended add-ons. Reproducible: Always Steps to Reproduce: 1. Open Add-ons window 2. Select Get Add-ons tab 3. Wait for recommended add-ons to appear Actual Results: The ratings icon shows the appropriate stars on the top recommended add-on, but shows a vertically-compressed composite of the entire image block in subsequent recommended add-ons. Expected Results: Each recommended add-on should show a single line of stars. This shows up both in the default theme and in my theme, which uses the older orange stars. This appears to be a regression.
The regression occurred between the 2008-03-18 and 2008-03-19 nightlies.
Assignee | ||
Comment 5•16 years ago
|
||
This is an expected regression from bug 423137. It will be resolved when AMO updates.
Status: UNCONFIRMED → RESOLVED
Closed: 16 years ago
Resolution: --- → INVALID
Assignee | ||
Comment 6•16 years ago
|
||
AMO has pushed their schedule back too far so we'll take a fix for this for b5.
Status: RESOLVED → UNCONFIRMED
Resolution: INVALID → ---
Assignee | ||
Comment 7•16 years ago
|
||
This ignores ratings < 0 and anything > 5 is constrained to be 5.
Assignee: nobody → dtownsend
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Assignee | ||
Comment 8•16 years ago
|
||
Attachment #310965 -
Attachment is obsolete: true
Updated•16 years ago
|
Flags: blocking-firefox3+
Priority: -- → P1
Target Milestone: --- → Firefox 3 beta5
Updated•16 years ago
|
Attachment #310970 -
Flags: review?(rstrong)
Updated•16 years ago
|
Attachment #310970 -
Flags: review?(rstrong) → review?(gavin.sharp)
Comment 9•16 years ago
|
||
Comment on attachment 310970 [details] [diff] [review] with unit tests <@gavin> r=me a=beltzner
Attachment #310970 -
Flags: review?(gavin.sharp)
Attachment #310970 -
Flags: review+
Attachment #310970 -
Flags: approval1.9b5+
Comment 10•16 years ago
|
||
Comment on attachment 310970 [details] [diff] [review] with unit tests Math.min(5, rating); would be clearer, I think.
Assignee | ||
Comment 11•16 years ago
|
||
Checking in toolkit/mozapps/extensions/src/nsAddonRepository.js; /cvsroot/mozilla/toolkit/mozapps/extensions/src/nsAddonRepository.js,v <-- nsAddonRepository.js new revision: 1.4; previous revision: 1.3 done RCS file: /cvsroot/mozilla/toolkit/mozapps/extensions/test/unit/test_bug424262.js,v done Checking in toolkit/mozapps/extensions/test/unit/test_bug424262.js; /cvsroot/mozilla/toolkit/mozapps/extensions/test/unit/test_bug424262.js,v <-- test_bug424262.js initial revision: 1.1 done RCS file: /cvsroot/mozilla/toolkit/mozapps/extensions/test/unit/data/test_bug424262.xml,v done Checking in toolkit/mozapps/extensions/test/unit/data/test_bug424262.xml; /cvsroot/mozilla/toolkit/mozapps/extensions/test/unit/data/test_bug424262.xml,v <-- test_bug424262.xml initial revision: 1.1 done
Attachment #310970 -
Attachment is obsolete: true
Attachment #310991 -
Flags: review+
Assignee | ||
Updated•16 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 16 years ago → 16 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Comment 12•16 years ago
|
||
Verified fix on Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9b5pre) Gecko/2008032304 Minefield/3.0b5pre. Star icons are back.
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
•