Make WordPress Core

Opened 8 years ago

Closed 7 years ago

#15480 closed defect (bug) (fixed)

JS sorting should set sorted arrow on both header and footer

Reported by: batmoo Owned by: scribu
Milestone: 3.1 Priority: normal
Severity: minor Version:
Component: Administration Keywords: has-patch ui-feedback
Focuses: Cc:


Currently, JS sorting only sets the sorted arrow for the context clicked (i.e. thead OR tfoot). The arrow should be added to both.

Attachments (2)

15480.diff (635 bytes) - added by batmoo 8 years ago.
15480_using_index.diff (688 bytes) - added by batmoo 7 years ago.
Improved version that uses index

Download all attachments as: .zip

Change History (7)

8 years ago

#1 @batmoo
8 years ago

  • Keywords ui-feedback added; ui removed

7 years ago

Improved version that uses index

#2 @batmoo
7 years ago

Updated alternate patch that's likely better since it uses the index of the header instead of matching based on text.

One inherent problem with the arrows, though, is that because we're using a CSS :hover, clicking to sort changes the arrow in the right direction, but the :hover kicks in and the arrow flips in the opposite direction. Unhovering reverts it back in the right direction. That should probably be fixed since it makes things confusing.

#3 @nacin
7 years ago

  • Milestone changed from Awaiting Review to 3.1

#4 @scribu
7 years ago

  • Owner set to scribu
  • Status changed from new to accepted

#5 @scribu
7 years ago

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

(In [16887]) Set both column header sorting arrows. Props batmoo. Fixes #15480

Note: See TracTickets for help on using tickets.