Closed
Bug 568077
Opened 14 years ago
Closed 14 years ago
Missing persona in the vertical carousel on Persona landing page
Categories
(addons.mozilla.org Graveyard :: Public Pages, defect, P1)
Tracking
(Not tracked)
VERIFIED
FIXED
5.11.1
People
(Reporter: krupa.mozbugs, Assigned: potch)
References
()
Details
(Whiteboard: [z][Personas])
Attachments
(1 file)
511.79 KB,
image/png
|
Details |
steps to reproduce: 1.Go to https://preview.addons.mozilla.org/z/en-US/firefox/personas/ 2.Notice that the vertical carousel has one missing persona see screenshot.
Updated•14 years ago
|
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → WORKSFORME
Reporter | ||
Comment 1•14 years ago
|
||
This is still valid.Just showed the behavior to Jeff.
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
Updated•14 years ago
|
Assignee: nobody → thepotch
Updated•14 years ago
|
Target Milestone: 5.11 → 5.11.1
Assignee | ||
Comment 2•14 years ago
|
||
This only happens when the user changes the zoom of the webpage, and then only when it is resized smaller. The easy fix is to hard-code the number of visible items, as the math that figures how many elements are visible at a time is what's being afffected by the resize.
Comment 3•14 years ago
|
||
Are there any overflow/underflow problems if it's hardcoded?
Assignee | ||
Comment 4•14 years ago
|
||
no impact other than making the component less re-usable, which makes me sad, but I will do in the name of it working better.
Updated•14 years ago
|
Priority: -- → P1
Assignee | ||
Comment 5•14 years ago
|
||
http://github.com/jbalogh/zamboni/commit/d03867d1
Assignee | ||
Updated•14 years ago
|
Status: REOPENED → RESOLVED
Closed: 14 years ago → 14 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 6•14 years ago
|
||
verified at https://preview.addons.mozilla.org/en-US/firefox/personas/
Status: RESOLVED → VERIFIED
Updated•8 years ago
|
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•