Skip to content

Conversation

@geriux
Copy link
Contributor

@geriux geriux commented Dec 1, 2021

This PR fixes a bug where some block-based themes that have font-size values with decimals like 1.125rem would crash the editor.

To test check the Gutenberg PR description.

PR submission checklist:

  • I have considered adding unit tests where possible.
  • I have considered if this change warrants user-facing release notes more info and have added them to RELEASE-NOTES.txt if necessary.

@guarani
Copy link
Contributor

guarani commented Dec 7, 2021

👋 Hi @geriux. We will cut the 1.68.0 release on Dec 9th. I plan to circle back and bump this PR to the next milestone then, but please let me know if you’d rather us work to include this PR in 1.68.0. 🙇

@geriux
Copy link
Contributor Author

geriux commented Dec 9, 2021

please let me know if you’d rather us work to include this PR in 1.68.0

Hey there @guarani 👋 Yes, I'd like this PR to be included, I'm working on merging it now 😃

@peril-wordpress-mobile
Copy link

Wanna run full suite of Android and iOS UI tests? Click here and 'Approve' CI job!

Copy link

@antonis antonis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The reference change LGTM 🎉
Thank you 🙇

@geriux geriux merged commit bcfe6d3 into develop Dec 9, 2021
@geriux geriux deleted the fix/font-size-parsing branch December 9, 2021 11:42
@guarani guarani mentioned this pull request Dec 9, 2021
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants