Twenty Twenty-Three: Inline CSS - Unit of measure is redundant

[ankitmaru]

On a few pages, margin properties have not proper properties as per standards.

CS: Unit of measure is redundant

Inline CSS fixes on a few templates.

Trac ticket: https://core.trac.wordpress.org/ticket/58485

This Pull Request is for code review only. Please keep all other discussion in the Trac ticket. Do not merge this Pull Request. See ​GitHub Pull Requests for Code Review in the Core Handbook for more details.

[SergeyBiryukov]

In 55898:

Twenty Twenty-Three: Remove redundant px unit in a few inline styles.

Follow-up to [54312], [54629].

Props ankitmaru.
Fixes #58485.

Thanks for the PR! Merged in r55898.

[TobiasBg]

I haven't tested this, but couldn't this trigger a "block content has changed" warning/error on existing sites? Or wait, rather wouldn't the extra units be coming back as they are still part of the block attributes?

[poena]

Hi

This does not cause any validation errors for the block markup in the editor, but I would prefer for it to be reverted:

1) The values in the block comment and the style attribute needs to match. It did not cause a block validation this time, but if it had been a different block attribute like class name or tag name, the mismatch would have caused an error.

2) The block control does not accept unitless values. If we would solve the mismatch and remove the unit from both the comment and the markup, then the block control
in the editor would not display the correct value. I believe this may be why the unit is not removed by the editor when it prepares the CSS.

[oglekler]

@SergeyBiryukov can you address the comment from Carolina?

I browsed the code, and it looks like 0px or 0rem in TwentyTwo theme all over the place, but it is only there. And here is a good point that block attributes and actual block needs to be match. 🤔

[oglekler]

This ticket was discussed during a bug scrub, and comment from @audrasjb: "Given we’re so close to the RC cycle, let’s revert it for now, then move it to milestone 6.4"

[audrasjb]

As per today's bug scrub: self assigning for revert.

[audrasjb]

In 56253:

Twenty Twenty-Three: Revert [55898].

This changeset reverts [55898] which removed redundant px units in a few inline styles, because 1) The values in the block comment and the style
attribute needs to match ; 2) The block control does not accept unitless values.

Follow-up to [54312], [54629], [55898].

Props TobiasBg, poena, oglekler.
See #58485.

[audrasjb]

Moving for 6.4 consideration.

[marybaum]

From September 5 scrub: we'll test the patch, which I'm sure is perfect, and then maybe next week or sooner we can refer for commit.

[nicolefurlan]

@audrasjb do we want to do anything here or should this ticket be closed? It looks like we originally committed the revision to remove the px units, then reverted that change, which is how things stand now.

[nicolefurlan]

This ticket was discussed in today's bug scrub.

Closing this ticket as wontfix, as the original fix was reverted and it seems like we don't need to pursue any further changes here. Per #comment:6, the px units are needed and intentional by design.

Props to @hellofromTonya :)