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

fix(locales): "Show More"の日本語訳を修正 #11073

Closed

Conversation

ryohidaka
Copy link
Contributor

#10747

What

以下のIssueのフォローアップです。

Why

Issueとして上がっていたため

Additional info (optional)

  • 内容としてはローカライゼーションに当たると思われるので、ターゲットブランチをl10n_developにしています。

  • 関西弁(ja-KS)に関しては、現状どちらもまだまだあるで!となっておりますが、代案が思いつかなかったのでそちらは変更していません。

  • CHANGELOG.mdも更新していませんが、必要でしたら更新分をコミットするので、コメント等で教えてください。

Checklist

  • Read the contribution guide
  • Test working in a local environment
  • (If needed) Add story of storybook
  • (If needed) Update CHANGELOG.md
  • (If possible) Add tests

@codecov
Copy link

codecov bot commented Jul 2, 2023

Codecov Report

Merging #11073 (fc733e0) into l10n_develop (3066a13) will increase coverage by 8.22%.
The diff coverage is n/a.

@@               Coverage Diff                @@
##           l10n_develop   #11073      +/-   ##
================================================
+ Coverage         69.14%   77.37%   +8.22%     
================================================
  Files               706      908     +202     
  Lines             65339    91292   +25953     
  Branches           5186     6842    +1656     
================================================
+ Hits              45179    70637   +25458     
- Misses            20160    20655     +495     

see 570 files with indirect coverage changes

@syuilo
Copy link
Member

syuilo commented Jul 6, 2023

基本的にローカリゼーションはMisskeyのドメイン知識に依存しない方がよく、そうするとShow moreの訳として「もっと見る」は正しい

@syuilo
Copy link
Member

syuilo commented Jul 6, 2023

なので他の解決方法を探す必要があるためこちらはCloseします🙏

@syuilo syuilo closed this Jul 6, 2023
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

2 participants