Opened 4 weeks ago
Last modified 5 days ago
#41969 assigned task (blessed)
Ensure Gallery widget is styled properly across widget areas in bundled themes
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | 4.9 | Priority: | high |
| Severity: | normal | Version: | |
| Component: | Bundled Theme | Keywords: | needs-patch ui-feedback |
| Focuses: | Cc: |
Description
The Gallery widget (#41914) allows for image galleries to be added to widget areas. Many themes may not have styles in them that expect galleries to appear in a widget context, since previously galleries would have only appeared in post content. As noted in §Default Themes Updates for Media Widgets for Images, Video, and Audio:
Themes that add custom styles to the MediaElement.js player (namely Twenty Thirteen and Twenty Fourteen) were updated from just styling it within syndicated content, to also include instances within widgets. Most themes don’t restrict styles for captioned images or media players to just post content, that is, limit CSS selectors to classes output by post_class(). If your theme does, make sure to either remove that constraint or include a .widget selector.
In the same way for the Gallery widget, we need to make sure that all of the core themes are compatible. Gallery widgets need to be tested in all core themes and in each of their widget areas, while also changing settings like the number of columns and the image size, in order to check for the variations.
Once the core themes are updated then the changes will may be instructive for theme authors to make similar changes in their themes.
Attachments (11)
Change History (21)
#1
@westonruter
4 weeks ago
- Keywords needs-patch added
- Owner set to davidakennedy
- Status changed from new to assigned
#2
@melchoyce
4 weeks ago
- Keywords needs-screenshots ui-feedback added
This ticket was mentioned in Slack in #design by karmatosed. View the logs.
4 weeks ago
#4
@joyously
4 weeks ago
- Keywords needs-screenshots removed
I added screenshots. They mostly look good, but Twenty Thirteen has some obvious problems, and Twenty Ten has a little overlap.
#5
@westonruter
11 days ago
- Type changed from defect (bug) to task (blessed)
See similar task for media added to Text widget: #42203
This ticket was mentioned in Slack in #core-themes by westonruter. View the logs.
11 days ago
#7
@williampatton
10 days ago
A little bit of CSS can fix some of the issues shown in twentythirteen theme. It doesn't fix them all (larger amounts of columns with non-thumbnail image configs still have clearing issues) but it covers many of the cases.
I will continue to work towards fixing any unusual quirks in the styling of these widgets across core themes as I have time.
This ticket was mentioned in Slack in #core by melchoyce. View the logs.
7 days ago
This ticket was mentioned in Slack in #core by davidakennedy. View the logs.
5 days ago
#10
@davidakennedy
5 days ago
Thanks @joyously for the testing and screenshots! Thanks @williampatton for the testing and patch!
I've been testing this today, and am seeing mostly the same stuff. I'm going to test a bit more tomorrow, check out the patch and see if anything else needs to be done.
Let's get some screenshots from each default theme, along with some style improvement suggestions :)