Skip to content
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

Dynamic locale doesn't localize element name #215

Closed
Tracked by #1004
oliver-phet opened this issue Oct 24, 2023 · 5 comments
Closed
Tracked by #1004

Dynamic locale doesn't localize element name #215

oliver-phet opened this issue Oct 24, 2023 · 5 comments

Comments

@oliver-phet
Copy link

This was reported by our Polish translator, but it appears to affect all locales.

Elements are translated properly in the Polish version of the sim, but not the all version(after dynamically switching locale).
image

May be related to #125 #90

@zepumph
Copy link
Member

zepumph commented Nov 16, 2023

Thanks for reporting, and I'm sorry it took me so long to get to this. It looks like I totally omitted supporting dynamic locale with ShredStrings. It was easy enough to implement above, and now we can MR it into BAN 1.1. Thanks for your patience.

@zepumph
Copy link
Member

zepumph commented Nov 16, 2023

QA: The bug was that the element name/identifier for a nuclide was not supporting dynamic local. You can see the bug here:

https://phet.colorado.edu/sims/html/build-a-nucleus/1.1.0/build-a-nucleus_all.html

Please make sure that switching the locale from the preferences menu updates the element name as well. Feel free to close.

@Nancy-Salpepi
Copy link

This looks fixed in 1.1.1-rc.1. Closing.

@zepumph
Copy link
Member

zepumph commented Nov 17, 2023

@oliver-phet this is fixed in https://phet.colorado.edu/sims/html/build-a-nucleus/1.1.1/build-a-nucleus_all.html, but please note that caching will not show the update for 24 hours:

https://phet.colorado.edu/sims/html/build-a-nucleus/latest/build-a-nucleus_all.html

Please let the user know. Thanks.

@oliver-phet
Copy link
Author

Thanks, translator notified!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants