![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png){kind=logo}
#43088 closed defect (bug) (fixed)
The media modal main background should be white
| Reported by: | afercia | Owned by: | SergeyBiryukov |
|---|---|---|---|
| Priority: | normal | Milestone: | 4.9.3 |
| Component: | Media | Version: | 4.9 |
| Severity: | normal | Keywords: | has-screenshots |
| Cc: | Focuses: | ui |
Description
In the media modal, the background of the content (the left part in the screenshots) used to be white:
now is gray:
Introduced in [41833].
When working on the admin stylesheets, low qualified selectors like .attachment-media-view tend to apply to several screens and should always be double checked.
Attachments (3)
Change History (10)
#1
@
9 years ago
Created patch 43088.diff to make media modal background white. It looks good to me!
#2
@
9 years ago
@chetan200891 thanks! You beat me by 9 seconds, lol.
However, the other backgrounds should stay gray (they're the "No image selected" placeholder in the widgets, see #42101).
#4
@
9 years ago
43088.3.diff tries a simpler approach to keep the gray background for all the widget placeholders and restore the white background in the media modal. Also, removes a trailing space.
Created patch to make media modal main background white.