Make WordPress Core

Opened 12 years ago

Closed 12 years ago

#28977 closed enhancement (fixed)

Media Grid missing 'View' link

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

Description

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

http://f.cl.ly/items/303J1X2Z2k352P472A3A/Media%20Library%20%E2%80%B9%20sadasd%20%E2%80%94%20WordPress%202014-07-21%2015-03-52%202014-07-21%2015-03-57.jpg

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 12 years ago.
add a View button to the media grid detail modal

Download all attachments as: .zip

Change History (8)

@adamsilverstein
12 years ago

add a View button to the media grid detail modal

#1 @adamsilverstein
12 years ago

  • Component GeneralMedia
  • 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:

http://f.cl.ly/items/1X0M2Z0p2V260c2X0u06/Media%20Library%20%E2%80%B9%20sadasd%20%E2%80%94%20WordPress%202014-07-21%2015-52-54%202014-07-21%2015-52-57.jpg

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

#2 @adamsilverstein
12 years ago

  • Summary Media missing 'View' linkMedia Grid missing 'View' link

#3 @SergeyBiryukov
12 years ago

  • Milestone Awaiting Review4.0

#4 @SergeyBiryukov
12 years ago

  • Owner set to SergeyBiryukov
  • Resolutionfixed
  • Status newclosed

In 29262:

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

props adamsilverstein.
fixes #28977.

#5 @helen
12 years ago

  • Resolution fixed
  • Status closedreopened

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

#6 @ericlewis
12 years ago

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

#7 @helen
12 years ago

  • Resolutionfixed
  • Status reopenedclosed

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.