-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Translation System, Settings Panel and more... #248
base: master
Are you sure you want to change the base?
Conversation
Some changes on i18n and on settings panel.
We need to talk about the UI :P |
Yes, sure. :P |
@gnuns Added some improvments to your PR (https://github.com/Gnuns/popcorn-app/pull/1) and it's not UI |
|
||
// checks if a locale is available | ||
i18n.isValidLocale = function i18nisValidLocale(key){ | ||
i18n.getLocales().indexOf(key) !== -1 |
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.
I forgot to add a return, it could break here. @gnuns can you add it for me ?
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.
done!
👍 |
Hey @Patineta, have you seen this? :X |
Thanks for the pull request! We've scheduled the settings panel for Beta 4, and since we're releasing Beta 2.5 today (with many hotfixes) we'll wait until then to add the settings module. |
Ok! |
Some improvements on i18n and on settings panel. An update of #209.
(Thanks @jeromelefeuvre)
I created a settings panel with some items requested on #150
To do:
(As you can see, i'm not a designer.)