WordPress.org
Get WordPress

Make WordPress Core

Opened 11 years ago

Closed 11 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:

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

Download all attachments as: .zip

Change History (8)

@adamsilverstein
11 years ago

add a View button to the media grid detail modal

#1 @adamsilverstein
11 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:

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 11 years ago by adamsilverstein (previous) (diff)

#2 @adamsilverstein
11 years ago

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

#3 @SergeyBiryukov
11 years ago

  • Milestone changed from Awaiting Review to 4.0

#4 @SergeyBiryukov
11 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
11 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
11 years ago

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

#7 @helen
11 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.