Posted by joshuajabbour on September 30, 2008 at 6:02pm
2 followers
| Project: | Nodequeue |
| Version: | 7.x-2.x-dev |
| Component: | User interface |
| Category: | task |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
Just a simple patch to sort available queues by title on the nodequeue admin page (admin/content/nodequeue/list). Currently nodequeues are listed in the order they are returned from the db, which *usually* appears to be by qid, but sometimes becomes semi-random.
This might be a candidate for moving deeper into the code, if there's a reason to return a list of queues sorted elsewhere, but I couldn't think of any. Additionally, I considered converting to a sortable table, but couldn't think of any real need to sort on a column besides title.
| Attachment | Size | Status | Test result | Operations |
|---|---|---|---|---|
| nodequeue.sortqueues.patch | 794 bytes | Ignored: Check issue status. | None | None |
Comments
#1
Thanks for another great usability improvement! Actually, I think using the core tablesort functionality could be useful here, mainly for sorting by the # of subqueues. Is that something you could re-roll?
#2
As you wish! Attached is a patch that adds the ability to sort on the first three columns.
I also thought about adding another column for nodequeue type (Taxonomy queue, Author queue, etc -- I've written a Node reference queue module I'll be posting soon), but we'd need to add a human-readable name to them (or display "smartqueue_taxonomy" etc). This can always be added later...
#3
Committed. This greatly improves the Node Queue administration screen. Thanks so much for accommodating this feedback!
#4
Automatically closed -- issue fixed for two weeks with no activity.