#56083 closed defect (bug) (fixed)
Twenty Twenty: The design of the ordered lists on the backend side is deteriorating
Reported by: |
|
Owned by: |
|
---|---|---|---|
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.
3 years ago
#2
Reference #56083
krupal-panchal commented on PR #2932:
3 years ago
#3
Can someone check this PR and give a comment?
@
11 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
@
11 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
@
11 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. Thank you everyone.
#8
@
11 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