Make WordPress Core

Opened 3 years ago

Closed 3 years ago

#22722 closed defect (bug) (fixed)

Create Gallery Shouldn't Say "All media"

Reported by: miqrogroove Owned by: koopersmith
Milestone: 3.5 Priority: normal
Severity: normal Version: 3.5
Component: Media Keywords:
Focuses: Cc:


I'm still extremely confused by the presence of the two different Library tabs in the new media modal. Here's a specific problem with them:

They both have an option called "All media items" that appear to do completely different things to the library view.

Attachments (2)

22722.diff (1.1 KB) - added by lessbloat 3 years ago.
22722.2.diff (904 bytes) - added by koopersmith 3 years ago.

Download all attachments as: .zip

Change History (15)

#1 @helenyhou
3 years ago

Galleries can only contain images, so I imagine that's what it's filtering. Perhaps the filter dropdown shouldn't say "All media items" but rather "Images".

It is actually much better for the user that they can only select items that can actually go into a gallery. Would be much worse if they could select audio or PDFs that won't show in the end result.

3 years ago

#2 @lessbloat
3 years ago

22722.diff​ switches "All media items" to "All images" in the Gallery view.

Version 0, edited 3 years ago by lessbloat (next)

#3 @koopersmith
3 years ago

Two things. First, we should probably use just "Images", as the "add to gallery" library only shows images not in the gallery (so, not *all* images) and "All images" is a new string. Second, this view is not tailored for images alone (it just so happens that we use it with image queries), so "all images" may be inaccurate if applied directly to the view.

3 years ago

#4 @koopersmith
3 years ago

  • Keywords has-patch needs-testing added

attachment:22722.2.diff uses the correct string based on the query's mime type and whether the mime type string is available.

#5 @lessbloat
3 years ago

  • Keywords needs-testing removed

+1 to 22722.2.diff​. Works well.

#6 @koopersmith
3 years ago

  • Keywords commit added

#7 @nacin
3 years ago

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

In 23041:

When adding images to a gallery, use "Images" as the filter name, rather than "All media items". props koopersmith. fixes #22722.

#8 @pavelevap
3 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

Filter name "Images" should be also used in "Add to Gallery" tab for consistency?

#9 @nacin
3 years ago

  • Milestone changed from Awaiting Review to 3.5

#10 @nacin
3 years ago

  • Keywords has-patch commit removed

#11 @nacin
3 years ago

This should be fixed when #22723 is.

#12 @nacin
3 years ago

  • Owner changed from nacin to koopersmith
  • Status changed from reopened to assigned

#13 @markjaquith
3 years ago

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

In 23068:

Fix the "Uploaded to this post" filter in the "Add to gallery" state. fixes #22723

Also, use "images" wording instead of generic "media items" when in a gallery (images-only) context. fixes #22722

Both are props koopersmith

Note: See TracTickets for help on using tickets.