Opened 19 months ago
Last modified 8 months ago
#61138 new enhancement
Twenty Nineteen: Latest Posts block top/bottom spacing is not same on editor side and front-end side when we change its background.
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Milestone: | Future Release | Priority: | normal |
| Severity: | normal | Version: | 6.5 |
| Component: | Bundled Theme | Keywords: | needs-refresh 2nd-opinion |
| Focuses: | ui, css | Cc: |
Description
Hello,
I have reviewed "Latest Post" block and found that when we are trying to change it background color, Its text color and top/bottom space are not same on editor side & front-end side.
I think that it should be same on both sides(Editor & Front-end)
Here, I have attached its screenshots:
Thanks,
Attachments (5)
Change History (8)
@
19 months ago
I have checked above mentioned issue and I have resolved it and added patch. Also, I have attached screenshot after resolved this issue.
#1
@
18 months ago
- Keywords needs-refresh added; dev-feedback removed
- Milestone changed from Awaiting Review to Future Release
I can confirm this is an issue. However, I think we need a different approach to the patch. First up there seem to be other things in this patch we might not need. Secondly, I think just leveling up the spacing is a gentler approach. We need to think that others might find any change a little unexpected so how can we do this as small as possible.
#2
@
15 months ago
- Keywords 2nd-opinion added
- Summary changed from Twenty Nineteen: Latest post block text color and top/bottom spacing are not same on editor side and front-end side when we change its background. to Twenty Nineteen: Latest Posts block top/bottom spacing is not same on editor side and front-end side when we change its background.
- Type changed from defect (bug) to enhancement
Let's keep the discussion about automatic contrast text colors with background on #49931.
For the padding, WordPress 6.6 fixed the discrepancy between the editor and front. The lower :root :where() specificity stopped overriding the theme's setting of zero padding for the Latest Posts block in the editor. On the front, the theme's specificity already was higher.
Editor:
/* Twenty Nineteen style-editor.css */
.wp-block-latest-posts {
padding: 0;
}
/* block-library/style.css */
:root :where(ul.has-background, ol.has-background) {
padding: 1.25em 2.375em;
}
/* block-library/editor.css */
:root :where(.wp-block-latest-posts) {
padding-left: 2.5em;
}
:root :where(.wp-block-latest-posts.is-grid), :root :where(.wp-block-latest-posts__list) {
padding-left: 0;
}
Front:
/* Twenty Nineteen style.css */
.entry .entry-content .wp-block-latest-posts {
padding: 0;
}
/* block-library/style.css */
:root :where(.wp-block-latest-posts.wp-block-latest-posts__list) {
padding-left: 0;
}
:root :where(ul.has-background, ol.has-background) {
padding: 1.25em 2.375em;
}
The block might look better with a small amount of padding on all sides whenever it has a background color, but that would be an enhancement. *If* this is a good idea, I would suggest 0.5rem on the front and 11px (0.5 * $font__size_base) in the editor.
#3
@
8 months ago
I attempted to replicate the issue using 6.9-alpha-60093-src, but I was unable to reproduce it. Based on my testing, the issue no longer appears to be present.
For reference, here is the video: https://rioudcpuyg.ufs.sh/f/PL8E4NiPUWyO72w7F19AWOSNT8iuxYqzcvCVIEKbQmF3njPo
Please let me know if any additional checks are needed.
Back-end