Ticket #4805 (closed enhancement: fixed)

Opened 11 months ago

Last modified 9 months ago

Use jQuery for WP's list manipulation

Reported by: mdawaffe Assigned to: mdawaffe
Priority: normal Milestone: 2.5
Component: Administration Version:
Severity: normal Keywords: jquery javascript listman i18n early
Cc:

Description

All that adding, deleting, and "dimming" stuff we do with Prototype should be rewritten for jQuery.

And here it is.

Attached is a patch that reworks the list manipulation stuff to be more markup based than JS based. Specific classes and ids are given to lists, list items, deletors, dimmers and adders which the wpLists jQuery plugin then finds and initializes.

Since a complete description of the markup involved would be rather lengthy, I have also attached a plugin (dependent on files found in the patch) that adds a Top Level "WPLists" menu item with documentation and examples.

The tests from the plugin work in the Firefox, Safari and Opera.

WordPress itself uses WPLists in fairly complicated ways is some places. Currently, those complicated pieces have only been tested in Firefox.

To Do:

  1. Test in IE
  2. Bring back JS "Are You Sures?". This will require #4459 to be done "properly".
  3. Bring back success messages ("X added").

Attachments

4805.diff (75.0 kB) - added by mdawaffe on 08/23/07 23:14:43.
Big Patch
test-wplists.php (17.2 kB) - added by mdawaffe on 08/23/07 23:15:37.
Plugin: Docs, Tests/Examples
4805b.diff (75.1 kB) - added by mdawaffe on 10/03/07 23:30:43.
rev 6185
4805c.diff (75.1 kB) - added by mdawaffe on 10/04/07 05:42:28.
rev 6185

Change History

08/23/07 23:14:43 changed by mdawaffe

  • attachment 4805.diff added.

Big Patch

08/23/07 23:15:37 changed by mdawaffe

  • attachment test-wplists.php added.

Plugin: Docs, Tests/Examples

08/24/07 15:48:44 changed by mdawaffe

Item 2 should read: #4559

09/13/07 04:10:20 changed by mdawaffe

  • keywords changed from jquery javascript listman i18n to jquery javascript listman i18n early.

10/03/07 23:30:43 changed by mdawaffe

  • attachment 4805b.diff added.

rev 6185

10/03/07 23:32:10 changed by mdawaffe

  • status changed from new to assigned.

4805b.diff

  1. Not tested much since last upload. Just refreshes the patch.
  2. Download and install attached test-wplists.php plugin after applying patch. Documents syntax and provides some tests.

10/04/07 04:40:32 changed by mdawaffe

Apparently I hardcoded the blog URL in the JS file. I'll have a better patch up in a bit.

10/04/07 05:42:28 changed by mdawaffe

  • attachment 4805c.diff added.

rev 6185

10/09/07 22:49:43 changed by ryan

  • status changed from assigned to closed.
  • resolution set to fixed.

(In [6213]) jquery for list manipulation. Round one, fight. Props mdawaffe. fixes #4805

10/09/07 22:51:26 changed by ryan

(In [6214]) jquery for list manipulation. Round one, fight. Props mdawaffe. fixes #4805