-
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
WP_Theme_JSON_Resolver: update translate terminology #28944
Conversation
Size Change: 0 B Total Size: 1.37 MB ℹ️ View Unchanged
|
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.
Code looks good. I don't get any errors in the log and everything seems to be working fine on the frontend 👍
8fbd28f
to
ce74abd
Compare
I see there's reported by rnmobile-android-runner https://github.com/WordPress/gutenberg/actions/runs/558545439 It's unrelated to these changes so I'm going to merge to continue work on #28783 |
While working on adding a new path to translate from the
theme.json
at #28783 I found myself wanting to do some refactorings prior. I thought it'd be easier to review if we tidy first, and then make the behavioral changes after.How to test
This PR doesn't introduce any behavioral change, so it's enough to activate TT1-blocks theme and load the front-end and editors. See that everything works as expected.