-
Notifications
You must be signed in to change notification settings - Fork 4.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
components: Remove wp-g2 fontsize control #31287
Conversation
Size Change: -82.7 kB (-6%) ✅ Total Size: 1.31 MB
ℹ️ View Unchanged
|
Nice 30% gain in the components package :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess this is the last thing that was importing external g2
components and that we could remove them now from package.json?
I was going to do it in a separate follow up PR 😁 |
With -82.7 kB, we should be back to where we were before 😄 From #27594 (comment): |
Description
Removes the wp-g2 font size control and reverts back to the previous implementation. This isn't permanent but gets us a step closer towards removing the wp-g2 dependencies in favor of vanilla emotion. See #30503 for the full plan and how this PR fits into it.
How has this been tested?
Run Gutenberg and ensure that the font size controls work (for example on the Paragraph block).
Screenshots
Types of changes
Non-breaking change/wp-g2 removal.
Checklist:
*.native.js
files for terms that need renaming or removal).