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
Issue 1599 - add --lang option to guide #1626
Conversation
Hi @ijlee2 please review, thanks in advanced. |
@@ -31,10 +31,11 @@ you will have access to a new `ember` command in your terminal. | |||
You can use the `ember new` command to create a new application. | |||
|
|||
```bash | |||
ember new ember-quickstart | |||
ember new ember-quickstart --lang en |
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.
Can we update the example language to en-US
, to better match the one in the ember-new-lang RFC? I think this may also help distinguish the locale from en-UK
, for example.
ember new ember-quickstart --lang en | |
ember new ember-quickstart --lang en-US |
(feel free to ignore this suggestion)
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.
Actually, the RFC does mention the example of just --lang en
. I think what you have currently is okay. 👍🏼
``` | ||
|
||
This one command will create a new directory called `ember-quickstart` and set up a new Ember application inside of it. | ||
The `--lang en` option sets the app's primary language to English to help improve [accessibility](../../accessibility/application-considerations/). |
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.
The `--lang en` option sets the app's primary language to English to help improve [accessibility](../../accessibility/application-considerations/). | |
The `--lang en-US` option sets the app's primary language to (US) English to help improve [accessibility](../../accessibility/application-considerations/). |
(feel free to ignore this suggestion)
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.
@fozy81 Thank you for working on updating the Application Considerations page to show the --lang
option. I think the PR is good to be merged!
fixes issue #1599 -does this wording look okay? please review.