-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Translate nav into ja #893
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
Conversation
nav_search_placeholder: "検索", | ||
nav_search_aria: "TypeScript サイト内を検索", | ||
nav_beta: "Search the TypeScript site", | ||
nav_beta_notification: |
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.
It looks we no longer used both nav_beta
and nav_beta_notification
on the TopNav
.
Can I remove them ?
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.
Yes!
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.
I'm going to intentionally close and re-open this PR, to see if https://github.com/orta/code-owner-self-merge works |
Looks like it only affects new PRs 👍🏻 |
Let's get this in, I'd rather it get merged than stay idle for too long |
Part of #220
Navigation bar translation.
nav