Make WordPress Core

Opened 10 years ago

Closed 10 years ago

#28977 closed enhancement (fixed)

Media Grid missing 'View' link

Reported by: adamsilverstein's profile adamsilverstein Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 4.0 Priority: normal
Severity: normal Version: 4.0
Component: Media Keywords: has-patch dev-feedback
Focuses: javascript Cc:


In the list view, its easy to view a media attachment, just hover over the row and click the View link.

The grid view should have a similar link, probably a 'View' button in the detail modal.

Attachments (1)

28977.diff (1.5 KB) - added by adamsilverstein 10 years ago.
add a View button to the media grid detail modal

Download all attachments as: .zip

Change History (8)

10 years ago

add a View button to the media grid detail modal

#1 @adamsilverstein
10 years ago

  • Component changed from General to Media
  • Keywords has-patch dev-feedback added

in 28977.diff:

  • add a View button to the media grid detail modal

Input on the placement welcome!

Currently looks like this:

Last edited 10 years ago by adamsilverstein (previous) (diff)

#2 @adamsilverstein
10 years ago

  • Summary changed from Media missing 'View' link to Media Grid missing 'View' link

#3 @SergeyBiryukov
10 years ago

  • Milestone changed from Awaiting Review to 4.0

#4 @SergeyBiryukov
10 years ago

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

In 29262:

Media Grid: Add a View button to Attachment Details modal.

props adamsilverstein.
fixes #28977.

#5 @helen
10 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

Should not be a button, going to reopen to figure the UI out better.

#6 @ericlewis
10 years ago

We shouldn't add an event click callback for anchors, lest we reinvent the browser (see 28952)

#7 @helen
10 years ago

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

In 29263:

Move the "view" link for attachment details to the sidebar and use more specific language. Removes an unnecessary JS handler. fixes #28977.

Note: See TracTickets for help on using tickets.