Skip to content

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

Merged
merged 3 commits into from
Aug 21, 2020
Merged

Translate nav into ja #893

merged 3 commits into from
Aug 21, 2020

Conversation

bobstrange
Copy link
Contributor

Part of #220

Navigation bar translation.

@bobstrange bobstrange requested a review from orta as a code owner August 12, 2020 15:42
nav_search_placeholder: "検索",
nav_search_aria: "TypeScript サイト内を検索",
nav_beta: "Search the TypeScript site",
nav_beta_notification:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@orta

It looks we no longer used both nav_beta and nav_beta_notification on the TopNav.
Can I remove them ?

https://github.com/microsoft/TypeScript-Website/blob/v2/packages/typescriptlang-org/src/components/layout/TopNav.tsx#L101-L103

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@orta
Thank you 🙇 I've opened a PR to remove them. ( #898 ).

@orta
Copy link
Contributor

orta commented Aug 13, 2020

I'm going to intentionally close and re-open this PR, to see if https://github.com/orta/code-owner-self-merge works

@orta orta closed this Aug 13, 2020
@orta orta reopened this Aug 13, 2020
@orta
Copy link
Contributor

orta commented Aug 13, 2020

Looks like it only affects new PRs 👍🏻

@orta
Copy link
Contributor

orta commented Aug 21, 2020

Let's get this in, I'd rather it get merged than stay idle for too long

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.

2 participants