Closed
Bug 623714
Opened 13 years ago
Closed 13 years ago
Screenshot-related options are missing/ do not match mocks
Categories
(addons.mozilla.org Graveyard :: Developer Pages, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
5.12.6
People
(Reporter: krupa.mozbugs, Assigned: gkoberger)
References
()
Details
(Whiteboard: [Edit page])
Attachments
(1 file)
155.50 KB,
image/png
|
Details |
As per http://people.mozilla.com/~chowse/drop/amo/devtools/v3/manage_addon/03_edit_listing_expanded.png, users can do the following with their screenshots * Details * Move up * Move down * Delete Right now, we allow users to delete the screenshots. But the delete link does not looks similar to the mock up
Comment 1•13 years ago
|
||
Sorry I didn't realize there was a v3 mockup, I used the v1 mockup.
Reporter | ||
Comment 2•13 years ago
|
||
Updated•13 years ago
|
Summary: Screenshot related options are missing/ do not match mocks → Screenshot-related options are missing/ do not match mocks
Updated•13 years ago
|
Assignee: nobody → amckay
Assignee | ||
Updated•13 years ago
|
Assignee: amckay → gkoberger
Comment 3•13 years ago
|
||
now now, there's enough bugs for everyone to get one.
Assignee | ||
Comment 4•13 years ago
|
||
If I'm not mistaken (chowse?), the screenshot Krupa posted is outdated, too. http://people.mozilla.com/~chowse/drop/amo/devtools/v3/interaction/screenshot_video.png
Comment 5•13 years ago
|
||
Screenshot ordering is bug 622610, but we can change the delete text. If we have the size/dimensions/type data right there, we can add it, but it's not important. Size is most useful. We automatically convert everything to PNG when they upload, so that's kind of a throw away.
Comment 6•13 years ago
|
||
(In reply to comment #4) > If I'm not mistaken (chowse?), the screenshot Krupa posted is outdated, too. > > http://people.mozilla.com/~chowse/drop/amo/devtools/v3/interaction/screenshot_video.png You're correct. The UI in the interactions wireframe (w/ bottom-aligned metadata and the (x) remove icon) is the intended design. However, the 'Add Screenshot' button is fine, unless we add video support at a later date.
Assignee | ||
Comment 7•13 years ago
|
||
clouserw: I didn't do size/dimensions/type intentionally, since almost every single one of them will be "700x525" and "png" (and nobody will care about the size, since we're resizing them so there's nothing they can do about it). As for reordering, you can assign it to me.
Assignee | ||
Comment 8•13 years ago
|
||
https://github.com/jbalogh/zamboni/commit/cf206293df0090059590d67d7f384d3b5e1f7c1d
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
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
•