Make WordPress Core

Opened 19 years ago

Closed 19 years ago

#6011 closed defect (bug) (fixed)

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

Reported by: mdawaffe Owned by:
Priority: normal Milestone: 2.5
Component: General Version: 2.5
Severity: normal Keywords: has-patch commit
Cc: Focuses:

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 19 years ago.
6011b.diff (414 bytes ) - added by mdawaffe 19 years ago.

Download all attachments as: .zip

Change History (8)

@mdawaffe
19 years ago

#1 @ryan
19 years ago

  • Resolutionfixed
  • Status newclosed

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

#2 @lloydbudd
19 years ago

  • Resolution fixed
  • Status closedreopened

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
19 years ago

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

#4 @ryan
19 years ago

  • Resolutionfixed
  • Status reopenedclosed

@mdawaffe
19 years ago

#5 @mdawaffe
19 years ago

  • Keywords commit added
  • Resolution fixed
  • Status closedreopened

6011b.diff

Gets rid of JS error on edit-pages.php

#6 @ryan
19 years ago

  • Resolutionfixed
  • Status reopenedclosed

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

Note: See TracTickets for help on using tickets.