Opened 6 months ago
Closed 6 months ago
#22615 closed defect (bug) (fixed)
Responsive css for the media modal
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Priority: | low | Milestone: | 3.5 |
| Component: | Media | Version: | 3.5 |
| Severity: | normal | Keywords: | audit needs-patch |
| Cc: |
Description
The current goal is to make the new media modal work/look well on tablets and small-screen laptops (same as the rest of the admin).
Attachments (6)
Change History (19)
comment:3
koopersmith — 6 months ago
- Keywords dev-feedback added
Concur. Commit, mark as see, bump to low priority.
comment:4
koopersmith — 6 months ago
- Keywords dev-reviewed added; dev-feedback removed
...things that begin with "dev".
comment:5
koopersmith — 6 months ago
In 22887:
comment:6
koopersmith — 6 months ago
- Keywords has-patch commit dev-reviewed removed
- Priority changed from normal to low
- Keywords has-patch added
22615.3.patch improves the media selection in the toolbar.
koopersmith — 6 months ago
comment:8
koopersmith — 6 months ago
attachment:22615.4.diff builds off of attachment:22615.3.patch:
Needs RTL after the current patch on #22453 is committed.
- Tweaks media selection improvements, moves them to the earlier break point.
- Adjusts modal size, dashed line position, separator size, padding on sidebar, and menu size.
attachment:22615.5.diff adds a lot of RTL stuff. And refreshes 22615.4.diff. And looks good.
(Except that the search form isn't bumped quickly enough.)
comment:10
azaozz — 6 months ago
22615.5.diff looks good. The only thing I've noticed is the rounded search field in iOS, but that's the same as other search fields in the admin.
comment:11
ryan — 6 months ago
In 22960:
comment:12
nacin — 6 months ago
- Keywords audit needs-patch added; has-patch removed
This ticket is basically just open for any additional cleanups or adjustments we want to make; it's a rider.
comment:13
nacin — 6 months ago
- Resolution set to fixed
- Status changed from new to closed
If ocean90 or azaozz has anything else here, please re-open. Thanks.

In 22615-2.patch: