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
i18n Modal Fixes #254
i18n Modal Fixes #254
Conversation
✅ Deploy Preview for cheery-moxie-4f1121 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
tACK 3073919
Looks good today all are translated for me now.
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.
tACK 3073919
Abstract
Some modals and minor UI elements were not rendering the i18n lang, defaulting to English, like the Seed Phrase creation or "Encrypt/Change Pass" modals, now it renders using i18n.
This PR also removes what seems to be "duplicated" HTML, since the Seed Phrase Vue template already contains the entire modal, the
index.html
didn't need a clone of it.This PR also solves the
getParentLanguage
function to fallback to English if a parent code isn't supported by MPW, for ex: missing lines fromES-MX
would fallback toES
, but MPW does not haveES
, so would throw errors trying to load it, those lines will cascade down to English worst case scenario.Lastly, a typo in
negotiateLanguages
was fixed that probably broke some underlying library functionality.Testing
To test this PR, it's suggested to attempt these user flows, or variations of these:
ES-MX
loads correctly without errors.If any errors are found, the PR works unexpectedly, or you have viable suggestions to improve the UX or functionality of the PR, let me know!