Make WordPress Core

Opened 12 years ago

Closed 10 years ago

#24352 closed defect (bug) (fixed)

It is not obvious what does "Edit" mean in the info (bottom) part of the media selection window

Reported by: mark-k's profile mark-k Owned by:
Milestone: 4.1 Priority: normal
Severity: normal Version: 3.5
Component: Media Keywords: ux-feedback has-patch dev-feedback
Focuses: accessibility, javascript Cc:

Description

  1. Edit a post
  2. click "add media"
  3. select a media
  4. the bottom part of the window contains an "edit" link

It seems like Edit mode just lets the user rearrange the order of the selected media. Maybe it is my english but when I rearrange icons on my desktop I don't think about it as editing. The use of "Edit" in that context just feels strange and confusing.

Attachments (3)

Edit-selection-link.jpg (79.1 KB) - added by ericlewis 11 years ago.
29326.diff (592 bytes) - added by adamsilverstein 10 years ago.
change selection edit link to 'Edit selection'
24325.diff (592 bytes) - added by adamsilverstein 10 years ago.

Download all attachments as: .zip

Change History (16)

#1 @SergeyBiryukov
12 years ago

  • Keywords ux-feedback added

#2 @wonderboymusic
11 years ago

  • Keywords reporter-feedback close added

I'm having a hard time finding the "edit" link...

#3 @mark-k
11 years ago

  • Keywords reporter-feedback close removed

Maybe it was an omition, maybe things changed, but right now (3.8) you will get the edit link after selecting two images.

#4 @SergeyBiryukov
11 years ago

To clarify, the ticket is about the selection editing link: tags/3.8.1/src/wp-includes/media-template.php#L253.

#5 @ericlewis
11 years ago

It's a confusing part of the workflow here.

The "Edit" link is for reordering the selected attachments, either in the "Insert Media" or "Create gallery" workflows.

#6 @wonderboymusic
10 years ago

  • Focuses javascript added
  • Milestone changed from Awaiting Review to 4.1

The drag and drop doesn't work and "Edit" is beyond ambiguous

@adamsilverstein
10 years ago

change selection edit link to 'Edit selection'

#7 @adamsilverstein
10 years ago

  • Keywords has-patch dev-feedback added

in 24325.diff: change link from 'Edit' to' Edit selection' - drag and drop still needs work.

#9 follow-up: @ericlewis
10 years ago

Have we discussed getting rid of this part of the interface? I see it as more confusion-causing than problem-solving.

It is a UI element that's primary utility (removing attachments from the selection) is a duplicating functionality in the attachments browser . The secondary utility of it (reordering attachments) seems unuseful enough to jettison as well.

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

#10 in reply to: ↑ 9 @adamsilverstein
10 years ago

  • Focuses accessibility added

Replying to ericlewis:

Have we discussed getting rid of this part of the interface? I see it as more confusion-causing than problem-solving.

It is a UI element that's primary utility (removing attachments from the selection) is a duplicating functionality in the attachments browser . The secondary utility of it (reordering attachments) seems unuseful enough to jettison as well.

That seems reasonable, unless there is some back compat reason to keep it? One big concern is that the current main screen where you bulk select images (from post->insert media) is not accessible - its very difficult to select images and add/remove them from a group selection.

To see what I mean, try selecting a group of images with only your keyboard - selecting a group is pretty easy - one click on the first image, then a shift click on the last image to select a range. Now, try deselecting images - you can't! actioning the check/minus _removes_ the image (not the same) and actioning an image selects it, deselecting the rest.

#11 @johnbillion
10 years ago

r30685 missed this ticket.

#12 @johnbillion
10 years ago

In 30686:

Remove the sortable functionality from the selection editing toolbar at the bottom of the media manager.

Fixes #29934
See #24352
Props jessepollak

#13 @johnbillion
10 years ago

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

I'm going to close this as fixed following r30685 and r30686.

I think this section of the UI should stay, but if anyone thinks it should be removed (or at least removed from a given view) then open a new ticket.

Note: See TracTickets for help on using tickets.