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

Translate the index page to Japanese #907

Merged
merged 2 commits into from Jan 13, 2021

Conversation

swnakamura
Copy link
Contributor

Translated the index page.
@JohnTitor please review it!

Copy link
Contributor

@JohnTitor JohnTitor left a comment

Choose a reason for hiding this comment

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

Thanks! Left some comments, mainly about the title.

@@ -2,12 +2,12 @@
template = "edition-2/index.html"
+++

<h1 style="visibility: hidden; height: 0px; margin: 0px; padding: 0px;">Writing an OS in Rust</h1>
<h1 style="visibility: hidden; height: 0px; margin: 0px; padding: 0px;">RustでOSを書く</h1>
Copy link
Contributor

Choose a reason for hiding this comment

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

"Writing an OS in Rust" is the title and I'd like to remain as-is (like "the book" or "TRPL").

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think translating this would help the search engine bots, and make this blog reach more Japanese reader? (this text is hidden so only robots read it)

Copy link
Contributor

Choose a reason for hiding this comment

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

I thought people use "Writing an OS in Rust" for search anyway but realized it will show the original and a Japanese title will lead to more search results. I'm fine with this then 👍

blog/content/_index.ja.md Outdated Show resolved Hide resolved
Co-authored-by: Yuki Okushi <huyuumi.dev@gmail.com>
@swnakamura
Copy link
Contributor Author

@phil-opp translation and review is complete. Please merge this.

@phil-opp
Copy link
Owner

Thanks!

@phil-opp phil-opp merged commit 904d203 into phil-opp:master Jan 13, 2021
@swnakamura swnakamura deleted the translate_index_ja branch January 13, 2021 09:03
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.

None yet

3 participants