WordPress.org

Make WordPress Core

Opened 9 years ago

Last modified 9 years ago

#15580 closed defect (bug)

Fixes to Ajaxify Admin before 3.1 — at Version 14

Reported by: jane Owned by: scribu
Milestone: 3.1 Priority: highest omg bbq
Severity: blocker Version: 3.1
Component: Administration Keywords: has-patch commit
Focuses: Cc:
PR Number:

Description (last modified by scribu)

This ticket will hold the individual adjustments needed for the ajaxified admin features before we release for 3.1

  1. When using pagination located at the bottom of the table, should load new results and bump you to the top of the screen.
  1. On comments, no need to sort alphabetically by comment.
  1. On p=36220 comment screens, no need to sort by "in reply to" since they are all in reply to the same comment.
  1. Save-state hashes for URLs.

4b. Don't bump bulk action and other triggers back to the front page.

  1. Date columns should first sort by DESC, not ASC. Plugins will likely need a new way to request DESC over ASC for a specific column.

Change History (14)

#1 @nacin
9 years ago

  • Description modified (diff)
  • Keywords needs-patch added; ajax removed
  • Milestone changed from Awaiting Review to 3.1
  • Severity changed from normal to blocker

Adding a few more issues.

#2 @nacin
9 years ago

  • Priority changed from normal to highest omg bbq
  • Status changed from new to assigned

#3 @jane
9 years ago

Change default sort by chronology to be reversed by default (most recent first).

#4 @westi
9 years ago

  • Cc westi added

Also #15500

#5 @scribu
9 years ago

  1. On p=36220 comment screens, no need to sort by "in reply to" since they are all in reply to the same comment.

For clarity, the column is called "In response to" and I think it should simply not be shown at all.

#6 @scribu
9 years ago

It would be great if someone could take a look at http://core.trac.wordpress.org/ticket/14579#comment:158 as I don't have access to Internet Explorer.

#7 @scribu
9 years ago

(In [16588]) Make comment content column unsortable. See #15580

#8 @scribu
9 years ago

  • Description modified (diff)

#9 follow-up: @scribu
9 years ago

Date columns should first sort by DESC, not ASC.

Why?

#10 in reply to: ↑ 9 @nacin
9 years ago

Replying to scribu:

Date columns should first sort by DESC, not ASC.

Why?

Per Jane:

Change default sort by chronology to be reversed by default (most recent first).

This is the expected functionality. Oldest first is unintuitive.

#11 @scribu
9 years ago

Oldest first is unintuitive only if you're sorting by date, but there's only one date column.

When you're sorting by title, you don't expect to see the post with title Z first.

#12 @scribu
9 years ago

Ah, I now saw that it's only supposed to be applied for date columns. Sorry.

#13 @scribu
9 years ago

(In [16593]) Make date columns first sort in descending order. See #15580

#14 @scribu
9 years ago

  • Description modified (diff)
Note: See TracTickets for help on using tickets.