WordPress.org

Make WordPress Core

Opened 5 years ago

Closed 4 years ago

#19785 closed enhancement (fixed)

Uploaded Media Delete Confirm Buttons Not Aligned

Reported by: JarretC Owned by: markjaquith
Milestone: 3.4 Priority: normal
Severity: normal Version: 3.0
Component: UI Keywords: has-patch commit
Focuses: Cc:

Description

After uploading the media but before saving your changes if you choose to delete the file the Continue/Cancel buttons end up on a different line.

Just a small patch that adds a paragraph element around the buttons to add them onto the next line. This also spaces the Cancel button a little bit further away from the filename so that it isn't so close.

Attachments (8)

old_new.jpg (10.5 KB) - added by JarretC 5 years ago.
delete-confirm.patch (1.4 KB) - added by JarretC 5 years ago.
19785.delete-confirm.png (7.3 KB) - added by SergeyBiryukov 5 years ago.
With delete-confirm.patch
19785.patch (1.6 KB) - added by SergeyBiryukov 5 years ago.
19785.after.png (15.2 KB) - added by SergeyBiryukov 5 years ago.
With 19785.patch
delete-confirm-2.patch (1.8 KB) - added by JarretC 4 years ago.
19785.2.patch (1.4 KB) - added by lancewillett 4 years ago.
Clean up patch to remove CSS, just add the block-level element
19785.diff (1.3 KB) - added by aaroncampbell 4 years ago.

Download all attachments as: .zip

Change History (19)

@JarretC
5 years ago

#1 @linuxologos
5 years ago

  • Version changed from 3.3.1 to 3.0

#2 @DrewAPicture
5 years ago

Looks like an improvement. +1

#3 @jane
5 years ago

  • Keywords ui-feedback removed
  • Milestone changed from Awaiting Review to 3.4

I'm down.

@SergeyBiryukov
5 years ago

With delete-confirm.patch

@SergeyBiryukov
5 years ago

With 19785.patch

#4 follow-up: @SergeyBiryukov
5 years ago

One minor concern is that a <p> tag would take up a bit more space and mouse movement for short file names (see 19785.delete-confirm.png).

19785.patch moves the buttons to the next line for long file names only (see 19785.after.png).

#5 in reply to: ↑ 4 @JarretC
4 years ago

Replying to SergeyBiryukov:

One minor concern is that a <p> tag would take up a bit more space and mouse movement for short file names (see 19785.delete-confirm.png).

19785.patch moves the buttons to the next line for long file names only (see 19785.after.png).

True, I was just initially thinking that having it always under would provide a consistent place in the UI as it would never change, even though it is a minor change.

I agree with the spacing, was going to add a class to the <p> and shorten the space with CSS but was just wanting to get an initial response.

#6 @nacin
4 years ago

I would opt for a consistent location. Let's figure out the padding and commit this.

#7 @JarretC
4 years ago

Added in new patch with CSS class and CSS updates, put them in media-dev.css...not 100% sure if it is supposed to go there or not. If not, let me know!

http://core.trac.wordpress.org/attachment/ticket/19785/delete-confirm-2.patch

#8 @lancewillett
4 years ago

  • Keywords dev-feedback needs-testing added

@lancewillett
4 years ago

Clean up patch to remove CSS, just add the block-level element

#9 @aaroncampbell
4 years ago

  • Cc aaroncampbell added

I think adding the paragraph tag around the text instead of the images make more sense.

@aaroncampbell
4 years ago

#10 @aaroncampbell
4 years ago

  • Keywords commit added; dev-feedback needs-testing removed

Talked to Lance and decided that it did make more sense to put the paragraph tag around the text instead of the buttons. It has the same effect and it's more semantic. 19785.diff does just that.

#11 @markjaquith
4 years ago

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

In [20309]:

Wrap attachment delete link in <p> for semantics and alignment. props aaroncampbell, lancewillett. fixes #19785

Note: See TracTickets for help on using tickets.