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

Enhance Colletion page [vi] #103

Merged
merged 2 commits into from Feb 21, 2016
Merged

Enhance Colletion page [vi] #103

merged 2 commits into from Feb 21, 2016

Conversation

runlevel5
Copy link

There are many English terms that could not be 1:1 translated to Vietnamese. Most of the cases, translated terms make no senses whatsoever. Because most of Vietnamese developer uses English terms in their conversations, I don't see why we have to introduce awkward translated Vietnamese terms. I'd like to propose to keep the English word but introducing Vietnamese explanation clauses.

@runlevel5 runlevel5 changed the title Enhance Colletion page Enhance Colletion page [vi] Feb 20, 2016
@vnnoder
Copy link

vnnoder commented Feb 20, 2016

Agree with these changes. This seems to be more practical.

For those english terms that come with vietnamese explanation, should we only show explanation the first time the term appears in a lesson? I think most reader will read from the beginning to the end of a lesson and is already familiar with the term.

@runlevel5 runlevel5 added the enhancement An enhancement to Elixir School content label Feb 21, 2016
@runlevel5 runlevel5 self-assigned this Feb 21, 2016
@runlevel5
Copy link
Author

@vnnoder

should we only show explanation the first time the term appears in a lesson?

Yes and No, it depends on the context, some concepts are common in programming but some are unique in Elixir. Thus IMHO your suggestion would go well with the first.

@runlevel5
Copy link
Author

@trexnix thanks much for the comments, I've updated the PR

runlevel5 pushed a commit that referenced this pull request Feb 21, 2016
@runlevel5 runlevel5 merged commit 2644d94 into master Feb 21, 2016
@runlevel5 runlevel5 deleted the update-collections branch February 21, 2016 08:16
@brain-geek brain-geek mentioned this pull request Mar 14, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An enhancement to Elixir School content
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants