-
Notifications
You must be signed in to change notification settings - Fork 8.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
translate InfraOps visualization component (Part 1 - folder pages) #25194
translate InfraOps visualization component (Part 1 - folder pages) #25194
Conversation
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.
Here is the list of issues. Please through all files and fix by analogy in all files of this MR and in all files which you are going to create new MRs for.
💔 Build Failed |
…1 - folder components)
💔 Build Failed |
retest |
💔 Build Failed |
💚 Build Succeeded |
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.
Please look at my comments.
💚 Build Succeeded |
💚 Build Succeeded |
💚 Build Succeeded |
@@ -25,6 +26,8 @@ | |||
"src/ui/ui_render/bootstrap/app_bootstrap.js", | |||
"src/ui/ui_render/ui_render_mixin.js", | |||
"x-pack/plugins/monitoring/public/components/cluster/overview/alerts_panel.js", | |||
"x-pack/plugins/monitoring/public/directives/alerts/index.js" | |||
"x-pack/plugins/monitoring/public/directives/alerts/index.js", | |||
"x-pack/plugins/infra/public/utils/loading_state/loading_result.ts", |
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.
These files were excluded because i18n
validation tool can not parse them due to an issue in babel parser which is used by the tool.
💚 Build Succeeded |
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.
LGTM
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.
Thanks for slogging through this long list of labels. ❤️
Since you couldn't derive the semantics of the layouts from just reading this code, I tried to add some samples of ids below, which I hope would be more stable in future iterations of these pages. Let me know if they make sense and provide enough detail to extrapolate to the other labels I didn't comment on.
x-pack/plugins/infra/public/pages/link_to/redirect_to_node_logs.tsx
Outdated
Show resolved
Hide resolved
… intl.formatMessage() and directly wrap some classes by injectI18n()
💔 Build Failed |
retest |
💔 Build Failed |
💚 Build Succeeded |
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.
Thank you for accommodating my previous requests. I left just a few suggestions for fixes below.
💚 Build Succeeded |
💚 Build Succeeded |
…lastic#25194) * translate InfraOps vizualization component (Part 1 - folder pages) * update translation of Beats Management vizualization component (Part 1 - folder components) * Update toolbar.tsx * update translation of Beats Management vizualization component (Part 1 - folder components) * update translation of Infra Ops vizualization component (Part 1) * update translation of Infra Ops vizualization component (Part 1) * update Infra Ops Part 1 - change some ids, change i18n.translate() to intl.formatMessage() and directly wrap some classes by injectI18n() * update Infra-I - add static to displayName, update some ids
💚 Build Succeeded |
…25194) (#25936) * translate InfraOps vizualization component (Part 1 - folder pages) * update translation of Beats Management vizualization component (Part 1 - folder components) * Update toolbar.tsx * update translation of Beats Management vizualization component (Part 1 - folder components) * update translation of Infra Ops vizualization component (Part 1) * update translation of Infra Ops vizualization component (Part 1) * update Infra Ops Part 1 - change some ids, change i18n.translate() to intl.formatMessage() and directly wrap some classes by injectI18n() * update Infra-I - add static to displayName, update some ids
translate InfraOps visualization component (Part 1 - folder pages)
Issue #25523
Guidline
Readme
You can also set up a pseudo-locale to see what labels are translated. Pseudo-locale adds diacritical mark to each
[A-Za-z]
character and duplicates every third character to increase messages length by approx. 33%. It's also used to check whether ui breaks with increased message length or not.How to use:
Set
i18n.locale
toen-xa
in./config/kibana.yml
Restart
yarn start
.