#56083 closed defect (bug) (fixed)
Twenty Twenty: The design of the ordered lists on the backend side is deteriorating
Reported by: | aezazshekh | Owned by: | karmatosed |
---|---|---|---|
Milestone: | 6.6 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Bundled Theme | Keywords: | has-screenshots has-patch commit |
Focuses: | css | Cc: |
Description
When you set the ordered list style in the list style block and the list style number is more than 2 digits, the design of the text of the list on the backend side is deteriorating.
To understand more, I have added a video link of the issue below.
https://share.cleanshot.com/SNNV2FjnfkxQp5e92rf7
Attachments (11)
Change History (21)
This ticket was mentioned in PR #2932 on WordPress/wordpress-develop by thejaydip.
2 years ago
#2
Reference #56083
krupal-panchal commented on PR #2932:
2 years ago
#3
Can someone check this PR and give a comment?
@
8 months ago
edits padding rule so it does not apply to lists with a background, adds top margin to all list items in List blocks, and adds side margin rule for List Item blocks
#6
@
8 months ago
- Keywords needs-testing added
- Milestone changed from Awaiting Review to 6.6
- The 1.3em padding does not fit a List block that has a background, so I used the
:not()
selector to remove it in that case (added within:where()
to keep the same specificity). - List items need the top margin to match the front, and I targeted the List block so it would not apply to list-based blocks such as Archives or Latest Comments.
- The side margin overrides the margin in
edit-post/classic.css
, which has a less specific selector in the plugin this week than the rule below. I targeted only List Item blocks that are immediately inside the List block with a background (not nested lists)..editor-styles-wrapper .wp-block { margin-left: auto; margin-right: auto; }
#7
@
8 months ago
- Keywords commit added; needs-testing removed
I just tested the latest patch and can confirm that it does fit a background and also it fixes the issue. Based on this, I think adding makes sense.
The concern I have in adding this just to be clear is that others might have workarounds in their child themes but I think we can safely say this is a positive bug fix and something needed so add it. I am going to on that basis add the commit keyword after proving above the patch works in testing.
It's worth noting this also impacts ordered lists as per my screenshots attached for both list types.
Thank you everyone.
#8
@
8 months ago
I created a set of blocks to compare the lists to the Paragraph block alignment and to show nested lists.
Screenshot of Issue