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

Rename Contribute to Community in main nav #650

Merged
merged 2 commits into from
Feb 14, 2017
Merged

Rename Contribute to Community in main nav #650

merged 2 commits into from
Feb 14, 2017

Conversation

tysongach
Copy link
Contributor

No description provided.

locales/jp.yml Outdated
docs: Docs
logo: Middleman
search_placeholder: ドキュメント検索
Copy link
Member

Choose a reason for hiding this comment

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

Is this search_placeholder removal on purpose?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It’s not used anywhere.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Maybe it should be?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Looks like it was originally added long ago in c339021#diff-98e97cfa6efa32a2e18600bef3e4bc27R7, but wasn’t used when the search functionality changed.

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 put up a fixup commit which updates the template to use this translation: 61f6c8c

@tdreyno tdreyno merged commit f16391a into master Feb 14, 2017
@tdreyno tdreyno deleted the tg-community branch February 14, 2017 21:43
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