WordPress.org

Make WordPress Core

Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#4805 closed enhancement (fixed)

Use jQuery for WP's list manipulation

Reported by: mdawaffe Owned by: mdawaffe
Milestone: 2.5 Priority: normal
Severity: normal Version:
Component: Administration Keywords: jquery javascript listman i18n early
Focuses: 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 (4)

4805.diff (75.0 KB) - added by mdawaffe 7 years ago.
Big Patch
test-wplists.php (17.2 KB) - added by mdawaffe 7 years ago.
Plugin: Docs, Tests/Examples
4805b.diff (75.1 KB) - added by mdawaffe 7 years ago.
rev 6185
4805c.diff (75.1 KB) - added by mdawaffe 7 years ago.
rev 6185

Download all attachments as: .zip

Change History (10)

mdawaffe7 years ago

Big Patch

mdawaffe7 years ago

Plugin: Docs, Tests/Examples

comment:1 mdawaffe7 years ago

Item 2 should read: #4559

comment:2 mdawaffe7 years ago

  • Keywords early added

mdawaffe7 years ago

rev 6185

comment:3 mdawaffe7 years ago

  • 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.

comment:4 mdawaffe7 years ago

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

mdawaffe7 years ago

rev 6185

comment:5 ryan7 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

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

comment:6 ryan7 years ago

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

Note: See TracTickets for help on using tickets.