Skip to content
This repository has been archived by the owner on Oct 28, 2022. It is now read-only.

Undefined name for XX-Large font size setting #280

Closed
richtabor opened this issue Oct 13, 2022 · 11 comments
Closed

Undefined name for XX-Large font size setting #280

richtabor opened this issue Oct 13, 2022 · 11 comments
Milestone

Comments

@richtabor
Copy link
Member

I noticed a missing name value for the xx-large font size.

The others have a value from core (i.e. "Small", "Medium", "Large", Extra Large") — but xx-large is custom to Twenty Twenty Three, so it does not have a name.

CleanShot 2022-10-13 at 16 01 56@2x

@richtabor
Copy link
Member Author

This may be related (odd font size behavior in Marigold):

CleanShot 2022-10-13 at 16 21 27@2x

@carolinan
Copy link
Collaborator

We are past the string freeze for 6.1, I hope an exception can still be made.

@carolinan
Copy link
Collaborator

I do find it strange that it does not automatically fall back to for example the slug. I would expect the editor to handle a missing name.

@richtabor
Copy link
Member Author

Are the strings in theme.json font size name values translated?

@richtabor
Copy link
Member Author

I would expect the editor to handle a missing name.

Perhaps it should fallback to the slug value, if one is not provided.

@mikachan
Copy link
Member

I think XXL is a Core size, and I believe the 'undefined' issue should have been fixed by WordPress/gutenberg#44791, although I can also currently reproduce this with WP 6.1 RC1.

@richtabor
Copy link
Member Author

I don't see it here. Would it be elsewhere?

Yea I'm testing with RC1

@mikachan
Copy link
Member

The XXL size is listed here.

I've reported the issue here, and I'll see if I can find where this has broken between the above issue and now.

@mikachan mikachan added this to the RC 3 milestone Oct 17, 2022
@mikachan
Copy link
Member

I'm now seeing this on all font size presets defined by the theme, not just XXL. It looks like it happens when the theme does not supply a name key for the font size preset, even if it's an 'official' Core preset size.

I have a potential fix here: WordPress/gutenberg#45041.

@mikachan
Copy link
Member

This should be fixed with WordPress/gutenberg#45041, which will be included in WP 6.1 RC2 🎉

@mikachan
Copy link
Member

This is working in WP 6.1 RC2.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants