Opened 6 years ago
Closed 6 years ago
#45242 closed defect (bug) (fixed)
Twenty Thirteen: 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, has-screenshots, commit, dev-reviewed, fixed-5.0 |
Focuses: | Cc: |
Description
I’ve re-reviewed the Gutenberg support that added in #45041, 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 Thirteen, I found these issues:
- Centred widget lists (Categories, Archives…) need to have the text centred, to match the editor.
- 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.
- In the editor, column blocks are not displaying as ‘full’ or ‘wide’
- On the front-end, full and wide table blocks are narrow; they should span the full width, like in the editor.
- On the front-end,
wide
blocks seem a bit too wide in larger browser windows — this is due to the factfull
blocks max out at 1600px to fit in Twenty Thirteen’s design. Wide blocks should do something similar. - Editor width selectors can be simplified to just use
wp-block
in the Gutenberg 4.2 RC.
Attachments (14)
Change History (23)
#1
@
6 years ago
45242.patch fixes all of the above issues; I only included screenshots for specific updates where it seemed helpful.
#2
@
6 years ago
- Keywords commit added; needs-testing removed
@laurelfulford I gave this patch a spin and it looks good to go. Marking it as such.
#3
@
6 years ago
- Keywords dev-reviewed added
Same thing with the uncentered dot points as in Twenty Eleven, but otherwise, good to go.
#4
@
6 years ago
Thanks @davidakennedy & @pento!
45242.1.patch makes a couple small updates:
- Makes bullets in centred widget blocks be centred with the rest of the text.
- Fixes an issue @davidakennedy noted from the other patches: it makes sure the bottom margin on the last paragraph in the Media & Text block doesn't add a bunch of extra space underneath.