Make WordPress Core

Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#28915 closed defect (bug) (fixed)

Media Grid View: Delete link container should not be 100 percent width

Reported by: drewapicture's profile DrewAPicture Owned by: helen's profile helen
Milestone: 4.0 Priority: normal
Severity: normal Version:
Component: Media Keywords:
Focuses: ui Cc:

Description (last modified by DrewAPicture)

Maybe a bit of an edge case here, but I accidentally clicked the "Delete Permanently" link when clicking the whitespace in the Edit Image modal via the media grid layout. See attached for the focused link container to see what I mean.

I see the value in the various meta fields being able to expand to 100% width there, but in my opinion, only the actual 'Delete Permanently' label should be clickable.

Attachments (1)

delete_link_focused.png (16.9 KB) - added by DrewAPicture 10 years ago.

Download all attachments as: .zip

Change History (5)

#1 @DrewAPicture
10 years ago

  • Description modified (diff)

#2 @ericlewis
10 years ago

@helen is working on related layout in #28844

#3 @helen
10 years ago

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

In 29204:

Media Grid Attachment Details modal UI improvements:

  • Align better visually with the existing media modal and the post image edit modal.
  • Add back a link to the full attachment edit screen (post.php).
  • Add a title to the modal and move prev/next buttons next to the more-consistent close button.
  • Remove mode tabs (metadata vs. image editing) in favor of the Edit Image button.

Still to come: responsive, IE8 testing and targeted CSS (calc() usage), general CSS cleanup.

props ericlewis, helen, melchoyce. see #28844. fixes #28915.

#4 @SergeyBiryukov
10 years ago

  • Keywords needs-patch removed
  • Milestone changed from Awaiting Review to 4.0
Note: See TracTickets for help on using tickets.