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

【要望・提案】言語選択UIは国旗より文字列の方がよいかも #317

Closed
ryo-a opened this issue Jan 20, 2020 · 4 comments · Fixed by #324
Closed

【要望・提案】言語選択UIは国旗より文字列の方がよいかも #317

ryo-a opened this issue Jan 20, 2020 · 4 comments · Fixed by #324

Comments

@ryo-a
Copy link

ryo-a commented Jan 20, 2020

Qiitaの記事で本 repository のことを知りました。非常に素晴らしい企画だと思います:+1:

要望・提案を思いついた経緯

本題ですが、現時点では言語選択UIとして国旗(日本語:日の丸, 英語:イギリス国旗 , 中国語:中華人民共和国国旗)が採用されているようです。
しかしながら、Webサイト国際化においては言語と国を1対1で対応付けるのはあまり望ましくないとされています。

要望・提案の内容

言語選択についてはプルダウンメニューなど、国旗ではなく文字列での表記に変更すると望ましいと思いますが、いかがでしょうか。

補足

yarnpkg.com の例
image

electronjs.orgの例
image

@hyouchimaru
Copy link
Contributor

いい提案です!私もそう思います!

@ryo-a
Copy link
Author

ryo-a commented Jan 20, 2020

ありがとうございます!

また、現状のコードでは中国語が国コード(中華人民共和国)に基づく cn とされています。しかし、言語コードに基づくと zh が正しいので、差し支えなければこちらも合わせて修正したいところですが、どうでしょうか。

<ClickSpan
class="flag-icon flag-icon-cn"
onClick={setLanguage('cn')}
select={select === 'cn'}
/>

@ryamaguchi0220
Copy link
Contributor

ryamaguchi0220 commented Jan 20, 2020

言語選択UIの改善PRを作りました #324
言語コードについては、今回の件とは別問題なので混ぜずに別PRで処理するのが良いように思えますがどうでしょうか。基本的に「1 PR : 1 Issue」にした方が、PRの目的が明確になりレビューもやり易くて良いという意見です。

@ryo-a
Copy link
Author

ryo-a commented Jan 21, 2020

迅速な対応ありがとうございます!
#324 すごく良さそうです

そうですね、言語コードについては別 Issue を作成します。

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 a pull request may close this issue.

3 participants