Opened 6 months ago
Closed 6 months ago
#22722 closed defect (bug) (fixed)
Create Gallery Shouldn't Say "All media"
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | normal | Milestone: | 3.5 |
| Component: | Media | Version: | 3.5 |
| Severity: | normal | Keywords: | |
| Cc: |
Description
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)
Change History (15)
22722.diff switches "All media items" to "All images" in the Gallery, and the Featured Image view.
comment:3
koopersmith — 6 months 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.
koopersmith — 6 months ago
comment:4
koopersmith — 6 months 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.
comment:6
koopersmith — 6 months ago
- Keywords commit added
- Owner set to nacin
- Resolution set to fixed
- Status changed from new to closed
In 23041:
- Resolution fixed deleted
- Status changed from closed to reopened
Filter name "Images" should be also used in "Add to Gallery" tab for consistency?
comment:10
nacin — 6 months ago
- Keywords has-patch commit removed
comment:11
nacin — 6 months ago
This should be fixed when #22723 is.
comment:12
nacin — 6 months ago
- Owner changed from nacin to koopersmith
- Status changed from reopened to assigned
comment:13
markjaquith — 6 months ago
- Resolution set to fixed
- Status changed from assigned to closed
In 23068:

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.