Twenty Twenty: RTL list styles

[desrosj]

In #51574/[49510], styles were added to ensure proper nesting of nested unordered lists in the RTL style sheet.

This fixes the issue, but because the RTL stylesheets are built, the added style is removed after running npm run build.

To preserve the styles in the RTL stylesheet going forward, a reflective style needs to be added to the LTR stylesheet.

[mukesh27]

Hi!

There is no left padding in the frontend so do we need to add left padding in the frontend?

[sabernhardt]

52401.diff​ is fine, though the value could be adjusted from 2em to 1.3em for better consistency. The patch would keep those indentations for any sites using the next version of Twenty Twenty with either WordPress 5.5 or 5.6.

Reverting [49510] is another option. GB27947 fixed the underlying problem behind #51574 on the editor side for WordPress 5.7 (merged into trunk with #52394).

The left margin and padding added to these list elements was not reflected in RTL, and [49510] worked around that in the Twenty Twenty theme instead of correcting in the editor.

.editor-styles-wrapper ul,
.editor-styles-wrapper ol {
    margin-bottom: 28px;
    padding-left: 1.3em;
    margin-left: 1.3em;
}

Twenty Nineteen never had a theme-specific patch, so I'll close #51573.

[sabernhardt]

before patch in English, 5.7 trunk, Firefox/Windows

[sabernhardt]

after patch in English, 5.7 trunk, Firefox/Windows

[ryelle]

I think it makes sense to commit this patch so that the two files are synced up correctly, and so that Twenty Twenty can still support older WP versions.

[SergeyBiryukov]

In 50359:

Twenty Twenty: Make sure the RTL list styles are preserved after npm run build.

Adjust the padding value for better consistency.

Props desrosj, sabernhardt, ryelle, mukesh27.
Fixes #52401.