Make WordPress Core

Opened 17 years ago

Closed 17 years ago

#6011 closed defect (bug) (fixed)

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

Reported by: mdawaffe's profile mdawaffe Owned by:
Milestone: 2.5 Priority: normal
Severity: normal Version: 2.5
Component: General Keywords: has-patch commit
Focuses: 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 (2)

6011.diff (10.2 KB) - added by mdawaffe 17 years ago.
6011b.diff (414 bytes) - added by mdawaffe 17 years ago.

Download all attachments as: .zip

Change History (8)

@mdawaffe
17 years ago

#1 @ryan
17 years ago

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

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

#2 @lloydbudd
17 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

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

#3 @ryan
17 years ago

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

#4 @ryan
17 years ago

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

@mdawaffe
17 years ago

#5 @mdawaffe
17 years ago

  • Keywords commit added
  • Resolution fixed deleted
  • Status changed from closed to reopened

6011b.diff

Gets rid of JS error on edit-pages.php

#6 @ryan
17 years ago

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

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

Note: See TracTickets for help on using tickets.