-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Installer language and timezone #991
Installer language and timezone #991
Conversation
I like this, but I wonder if it might be simpler just using anticipate and autocomplete? Saves long lists... ? |
I see your point, however, the main idea of having a list was to limit the user from entering invalid choices. I split the timezone on two steps to limit the max length of the list. If it's really worth it to have a smaller list than this, we could do UTC +/- list, however i'd need to translate it to a timezone that TI understands. I could randomly choose one of the timezones that matches the UTC +/- but that can lead to confusion since there are multiple timezones with the same offset. |
To me if the dev enters an incorrect value thats on them... but I'll let @sampoyigi make the call |
Nice how you broke out the timezone into continents. I believe we can apply the anticipate and autocomplete approaches, and to avoid users entering invalid timezone/locale, we can validate user input and ask again if necessary. Not sure if its possible, what about an approach that detects the system locale and timezone and recommends using it during igniter:install? The user can either confirm using the system default settings or choose from the list. Can we also determine which locale to use based on the timezone? |
That's fine with me. |
Another thing, this PR should target the flame/v4 branch, the new destination for v4 stuff. |
As referenced in #904
Reopen of #908
Choosing the language
Split timezone selection:
Example choosing Africa
Example choosing Europe