Closed
Bug 471706
Opened 16 years ago
Closed 16 years ago
Editor Tools: add 'prev' and 'next buttons to review pages
Categories
(addons.mozilla.org Graveyard :: Admin/Editor Tools, enhancement)
Tracking
(Not tracked)
VERIFIED
FIXED
5.0.1
People
(Reporter: osunick, Assigned: wenzel)
Details
Attachments
(1 file)
6.46 KB,
patch
|
lorchard
:
review+
|
Details | Diff | Splinter Review |
When reviewing pending and nomination queues, it requires a lot of clicks to page through the add-ons for quick review. Can we throw in some navigation into those pages? It'd be great if we could get this done quickly as we start to bring on more heads on the editor team. Thanks!
Updated•16 years ago
|
Assignee: nobody → fwenzel
Target Milestone: --- → 5.0.1
Assignee | ||
Comment 1•16 years ago
|
||
Clarification: After reading discussions on amo-editors, I realized these navigation links are supposed to be on the individual addons' editor review pages, much link in bugzilla for individual bugs from a bug list.
Hardware: x86 → All
Reporter | ||
Comment 2•16 years ago
|
||
Exactly. Sorry if that wasn't clear earlier!
Assignee | ||
Comment 3•16 years ago
|
||
This patch adds a prev/next button to the review page. I am using the current queue position as a GET parameter, so I don't have to pass too much information around. Les, do you mind taking a look at this? Should be quite straightforward. Or, if you want to review it yourself, Nick, go ahead :)
Attachment #357680 -
Flags: review?(lorchard)
Assignee | ||
Updated•16 years ago
|
Status: NEW → ASSIGNED
Updated•16 years ago
|
Attachment #357680 -
Flags: review?(lorchard) → review+
Comment 4•16 years ago
|
||
Comment on attachment 357680 [details] [diff] [review] add prev/next buttons to review pages, rev 1 Might want to try getting the patch applied on khan to let Nick look at it, but works for me!
Assignee | ||
Comment 5•16 years ago
|
||
Nick: http://skitch.com/fredw/bb4b2/review-queue-nav-buttons
Reporter | ||
Comment 6•16 years ago
|
||
Perfect!
Assignee | ||
Comment 7•16 years ago
|
||
I replaced the strings with l10n tags and committed it to r21653. I am going to notify the localizers shortly.
Comment 8•16 years ago
|
||
Verified FIXED for en-US on https://preview.addons.mozilla.org/en-US/firefox/editors/review/57172?num=522; I checked both bounds and JavaScript-disabled views, additionally.
Status: RESOLVED → VERIFIED
Updated•15 years ago
|
Keywords: push-needed
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
•