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

docs: more info on translating #163

Merged
merged 6 commits into from Feb 8, 2019
Merged

Conversation

jakebolam
Copy link
Collaborator

@jakebolam jakebolam commented Feb 5, 2019

re #159.

@Berkmann18
Copy link
Member

Berkmann18 commented Feb 5, 2019

Is that inline with what needs to be done to resolve #159 or is it separate?
Just so I need to see if the changes would go in this PR or a different one?

@jakebolam
Copy link
Collaborator Author

@Berkmann18 I think it's ok to have multiple PRs for an issue. This change is totally not complete (just didn't want to lose the context I had when translating)

@Berkmann18
Copy link
Member

@jakebolam Okay, I'll try to draft something either tomorrow or sometime this week (as tomorrow is forecasted to be busy).

> When translating, its important that anything that has the [XPATH](https://developer.mozilla.org/en-US/docs/Web/XPath) `@href`, `@src` etc should not be translated. Things like `code` should not be either. This will cause the project to 404. See below image for an example:
<img src="../../assets/translating-xpath.png" alt="Contribute to translating" />

> However, it's recommended to translate any content that would contribute to a better comprehension like the comments in `code` tags _as long as_ it doesn't change the actual code since it only understands English.
Copy link
Member

Choose a reason for hiding this comment

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

@jakebolam What do you think about that?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Looks great!

Copy link
Member

@Berkmann18 Berkmann18 left a comment

Choose a reason for hiding this comment

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

LGTM.

@jakebolam jakebolam merged commit 3778594 into master Feb 8, 2019
@Berkmann18 Berkmann18 deleted the docs/document-translating-v-0 branch February 11, 2019 09:47
@all-contributors-release-bot
Copy link
Member

🎉 This PR is included in version 2.10.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

None yet

3 participants