Opened 6 years ago
Closed 6 years ago
#45240 closed defect (bug) (fixed)
Twenty Twelve: Issues in theme’s Gutenberg support & styles
Reported by: |
|
Owned by: |
|
---|---|---|---|
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 #45040, 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 Twelve, I found these issues:
- The placeholder text does not use the correct font styles.
- Centred widget lists (Categories, Archives…) need to have the text centred, like 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.
- Editor width selectors can be simplified to just use
.wp-block
in the Gutenberg 4.2 RC.
Attachments (10)
Change History (19)
#1
@
6 years ago
45240.patch should take care of all the issues listed above, though I only included screenshots where it seemed helpful.
#2
@
6 years ago
- Keywords commit added; needs-testing removed
@laurelfulford This patch looks good. I tested it, and don't see any other issues. Marking it good to go.
#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 @pento & @davidakennedy!
45240.1.patch makes a couple small updates:
- It makes sure bullets are centred when list-based widgets, like the Categories or Archives, are centred.
- It fixes an issue @davidakennedy noted in other themes, and removes the bottom margin from the last item in the Media & Text block.