Ticket #6011 (closed defect: fixed)

Opened 9 months ago

Last modified 9 months ago

No need to load wp-lists.js on many pages

Reported by: mdawaffe Assigned to: anonymous
Priority: normal Milestone: 2.5
Component: General Version: 2.5
Severity: normal Keywords: has-patch commit
Cc:

Description

Since the new UI does not have an interface for deleting individual posts, links, etc. (only bulk delete is offered), we don't need to load wp-lists.js on many of those pages.

Attached:

  1. Gets rid of wp-lists.js on pages that don't need it (edit-pages.php, link-manager.php, edit.php and edit-post-rows.php, upload.php).
  2. Gets rid of "infitite post deletion".
  3. Gets rid of edit-posts.js.
  4. Simplifies ajax adding of new users. Old UI had different lists for different roles. New UI only has one list.
  5. Unifies alternating row class (everything was "first row is alternate" except edit-tags.php which was "second row is alternate").

After applying patch: 

svn del wp-admin/js/edit-posts.js wp-admin/js/link-cat.js

(see [6213/trunk/wp-admin/js/link-cat.js] for that last.)

Attachments

6011.diff (10.2 kB) - added by mdawaffe on 02/27/08 00:18:59.
6011b.diff (414 bytes) - added by mdawaffe on 02/29/08 07:53:29.

Change History

02/27/08 00:18:59 changed by mdawaffe

  • attachment 6011.diff added.

02/27/08 00:46:27 changed by ryan

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

(In [7057]) Don't load wp-lists on pages that don't need it. Props mdawaffe. fixes #6011

02/27/08 03:25:25 changed by lloydbudd

  • status changed from closed to reopened.
  • resolution deleted.

ENV: trunk r7058

admin/js/edit-posts.js wp-admin/js/link-cat.js still present.

As above, after applying patch: 

svn del wp-admin/js/edit-posts.js wp-admin/js/link-cat.js

(see [6213/trunk/wp-admin/js/link-cat.js] for that last.)

02/27/08 05:05:28 changed by ryan

(In [7059]) Remove unused files. see #6011

02/27/08 06:21:55 changed by ryan

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

02/29/08 07:53:29 changed by mdawaffe

  • attachment 6011b.diff added.

02/29/08 07:53:54 changed by mdawaffe

  • keywords changed from has-patch to has-patch commit.
  • status changed from closed to reopened.
  • resolution deleted.

6011b.diff

Gets rid of JS error on edit-pages.php

02/29/08 08:12:13 changed by ryan

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

(In [7102]) Remove some no longer needed JS. Props mdawaffe. fixes #6011