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

Translation System, Settings Panel and more... #248

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Translation System, Settings Panel and more... #248

wants to merge 7 commits into from

Conversation

gnuns
Copy link
Contributor

@gnuns gnuns commented Mar 7, 2014

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
popc
To do:

  • Proxy settings
  • Improve "alwaysOnFocus" (it's using win.focus() for now)

(As you can see, i'm not a designer.)

Some changes on i18n and on settings panel.
@abadfederico
Copy link
Contributor

We need to talk about the UI :P

@gnuns
Copy link
Contributor Author

gnuns commented Mar 7, 2014

Yes, sure. :P

@jeromelefeuvre
Copy link
Contributor

@gnuns Added some improvments to your PR (https://github.com/Gnuns/popcorn-app/pull/1) and it's not UI :trollface:


// checks if a locale is available
i18n.isValidLocale = function i18nisValidLocale(key){
i18n.getLocales().indexOf(key) !== -1
Copy link
Contributor

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 ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done!

@jeromelefeuvre
Copy link
Contributor

👍

@gnuns
Copy link
Contributor Author

gnuns commented Mar 7, 2014

Hey @Patineta, have you seen this? :X

@Patineta
Copy link
Contributor

Patineta commented Mar 7, 2014

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.

@gnuns
Copy link
Contributor Author

gnuns commented Mar 7, 2014

Ok!

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

Successfully merging this pull request may close these issues.

4 participants