Opened 6 years ago
Closed 6 years ago
#45245 closed defect (bug) (fixed)
Twenty Sixteen: Issues in theme’s Gutenberg support & styles
Reported by: | laurelfulford | Owned by: | laurelfulford |
---|---|---|---|
Milestone: | 5.0 | Priority: | normal |
Severity: | normal | Version: | 5.0 |
Component: | Bundled Theme | Keywords: | has-patch, needs-testing, has-screenshots, dev-reviewed, fixed-5.0 |
Focuses: | Cc: |
Description
I’ve re-reviewed the Gutenberg support that added in #45044, primarily to find anything that might have been missed, but also make sure none of the other changes happening during 5.0 development so far created a need for theme updates, too.
I opted to combine my findings per theme into one ticket, since many of the issues showed up in more than one of the default themes — it seemed like it would make it easier to review and commit these fixes, given the timeline.
In Twenty Sixteen, I found these issues:
- Centred widget lists (Categories, Archives…) need to have the text centred, like the editor.
- When you centre the Cover block, the styles get messed up, with the text sitting at the top, and behind the overlay.
- The placeholder text uses the right font, but it’s inheriting a too-dark colour that makes it look less than placeholder-y.
- The cover blocks need to have their styles updated to add the newer non
-image-
variations of the cover block classes (.wp-block-cover
and.wp-block-cover-text
) - The Media & Text block does not have spacing underneath it.
- Editor width selectors can be simplified to just use
wp-block
in Gutenberg 4.2 RC 1. - The
margin
styles being pulled in from the theme’s original editor styles are causing the content in Gutenberg to list to the left. This is new in Gutenberg’s 4.2 RC 1 as well.
Attachments (14)
Change History (23)
#1
@
6 years ago
45245.patch includes fixes for all the issues listed above, but I only included screenshots in cases where it seemed like it would be helpful!
#2
@
6 years ago
Hey @laurelfulford – like Twenty Fourteen and Twenty Fifteen, this patch looks good, save for a similar issue with the media and text block. See: https://core.trac.wordpress.org/ticket/45243#comment:2 Otherwise, good to go.
#3
@
6 years ago
- Keywords dev-reviewed added
Same thing with the uncentered dot points as in Twenty Eleven, but otherwise, 👍🏻.
#4
@
6 years ago
Thanks @davidakennedy and @pento!
45245.1.patch makes a couple small changes:
- It removes the bottom margin from the last child in the media block.
- It updates the list styles of widgets, so when they're centred, the bullets display centred as well.