Skip to content

Comments

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

Closed
ryohidaka wants to merge 1 commit intomisskey-dev:l10n_developfrom
ryohidaka:fix/showmore-ja
Closed

fix(locales): "Show More"の日本語訳を修正#11073
ryohidaka wants to merge 1 commit intomisskey-dev:l10n_developfrom
ryohidaka:fix/showmore-ja

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

@github-actions github-actions bot requested review from acid-chicken and tamaina July 2, 2023 22:56
@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.

2 participants