Make WordPress Core

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#29104 closed defect (bug) (fixed)

Media Grid: Retain focus in modal when paginating

Reported by: ericlewis's profile ericlewis Owned by: wonderboymusic's profile wonderboymusic
Milestone: 4.0 Priority: normal
Severity: normal Version: 4.0
Component: Media Keywords: has-patch
Focuses: ui, javascript, administration Cc:

Description

In media grid, we blow away the attachment detail modal. Let's not do this, and just rerender the relevant parts of the template.

Attachments (1)

29104.diff (1.3 KB) - added by ericlewis 11 years ago.

Download all attachments as: .zip

Change History (4)

@ericlewis
11 years ago

#1 @ericlewis
11 years ago

  • Keywords has-patch added
  • Version set to trunk

In attachment:29104.diff, rerender the content region and toggle disabled states on the pagination buttons appropriately on pagination events.

#2 @wonderboymusic
11 years ago

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

In 29368:

Media Grid: don't blow away the Attachment Details modal every time a new model is loaded.

Props ericlewis.
Fixes #29104.

#3 @SergeyBiryukov
11 years ago

  • Milestone changed from Awaiting Review to 4.0
Note: See TracTickets for help on using tickets.