-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Interactivity Router: Pass translated strings via script module data #7304
base: trunk
Are you sure you want to change the base?
Interactivity Router: Pass translated strings via script module data #7304
Conversation
Test using WordPress PlaygroundThe changes in this pull request can previewed and tested using a WordPress Playground instance. WordPress Playground is an experimental project that creates a full WordPress instance entirely within the browser. Some things to be aware of
For more details about these limitations and more, check out the Limitations page in the WordPress Playground documentation. |
* @param array $data Data to filter. | ||
* @return array Data for the Interactivity Router script module. | ||
*/ | ||
public function filter_script_module_interactivity_router_data( array $data ): array { |
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.
Can't we find a better way?
This is really a step backwards from the JS I18N mechanism we implemented all those years ago 😞
Translators loose important context because the strings are no longer co-located with the code where they're being used. This also fails as soon as soon as you are dealing with plurals.
- Better modules - Parse wp modules from package.json - Combine asset file
This pulls the block-library assets from: WordPress/gutenberg@2632234
This reverts commit 404336b.
This script module requires an additional change that will be landed separately before it can be landed. See WordPress/gutenberg#65380.
…i18n-with-script-module-data
This is building on #7405. |
…i18n-with-script-module-data
This implements script module data passing for translated strings matching the behavior in WordPress/gutenberg#65123.
Related PRs:
Trac ticket:
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.