Opened 17 years ago
Closed 17 years ago
#6988 closed enhancement (fixed)
Add sorting to gallery items
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 2.6 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Administration | Keywords: | has-patch needs-testing blessed |
Focuses: | Cc: |
Description
The idea is to integrate the functionality that my Reorder Gallery plugin offers into core as per Matt Mullenweg's suggestion over at my blog.
It should be rather easy, I'll try to get a patch up real soon. Hopefully Matt will stop by and let me know what version he'd like to integrate it into.
Attachments (5)
Change History (23)
#4
@
17 years ago
The 002 diff is to be used instead of 001. It's 001 with the missing entry in script-loader for gallery.js (aliased as "admin-gallery").
#5
@
17 years ago
This adds an interface.js requirement. We're trying to eliminate interface requirements. Can jQuery UI be used instead?
#6
@
17 years ago
I'm sure it could, it was set up to mimic wp-admin/js/widgets.js which does the widget sorting. I'll take a look at jQuery UI in the next day or two and convert it.
#7
@
17 years ago
Yeah, widgets admin needs to be ported to jQuery UI. jQuery UI wasn't up to the task when 2.5 was released, but the latest jQuery UI beta might work.
#8
@
17 years ago
To use the jquery ui sortables, we need the ui.sortable file (duh), but we also have to add the latest jquery.js from trunk. Should I ADD it (tag it as jquery125), or replace the current one with it?
#11
@
17 years ago
003 is the newest update.
- It updates to the latest wp-includes/js/jquery/jquery.js from trunk (1.2.5 built from revision 5652)
- It updates to the latest wp-includes/js/jquery/ui.core.js from trunk (built from revision 5652)
- It adds wp-includes/js/jquery/ui.sortable.js (built from revision 5652)
- It adds wp-admin/js/gallery.js (the new version that uses UI)
- Adds the new stuff to script loader
- Adds in the new "menu_order" field
- Updates colors-fresh and colors-classic to use the same .sorthelper class for the widgets and the gallery.
- Updates media.css to add the "move" cursor to the elements
#12
@
17 years ago
My wikiformatting didn't work so great on that last post (left of a leading space before all my list items), so here it is:
Regarding 003:
- It updates to the latest wp-includes/js/jquery/jquery.js from trunk (1.2.5 built from revision 5652)
- It updates to the latest wp-includes/js/jquery/ui.core.js from trunk (built from revision 5652)
- It adds wp-includes/js/jquery/ui.sortable.js (built from revision 5652)
- It adds wp-admin/js/gallery.js (the new version that uses UI)
- Adds the new stuff to script loader 1. Adds in the new "menu_order" field
- Updates colors-fresh and colors-classic to use the same .sorthelper class for the widgets and the gallery.
- Updates media.css to add the "move" cursor to the elements
#14
@
17 years ago
- Resolution fixed deleted
- Status changed from closed to reopened
Seems to be broken now. Error in FB:
sort.instance is undefined http://site.com/wp26/wp-admin/js/gallery.js?ver=20080520 Line 13
I uploaded the patch, works on my trunk install.