misskey icon indicating copy to clipboard operation
misskey copied to clipboard

fix: #13368 最優先されるフォントをSegoe UIとHelveticaNeueにする

Open morotesovtannu opened this issue 1 year ago • 14 comments

What

優先するフォントを日本語フォントから欧文フォントに変更。 Vivaldi Snapshot(Windows 11)での変更結果は以下の通り。

Before OLD Profile image

After After Profile image After note image

Why

現行の日本語フォントを優先する設定だと、キリル文字がすべて全角で表示されてしまうため。 #13368

Additional info (optional)

HelveticaNeueをMac向けに優先させたが、現状Macでのテストは環境が無いためできていない。 fix #13368

Checklist

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

morotesovtannu avatar Feb 18 '24 14:02 morotesovtannu

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (b0030d1) 64.48% compared to head (921e90e) 64.46%.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #13390      +/-   ##
===========================================
- Coverage    64.48%   64.46%   -0.02%     
===========================================
  Files          981      981              
  Lines       109999   109999              
  Branches      5605     5593      -12     
===========================================
- Hits         70937    70915      -22     
- Misses       37628    37650      +22     
  Partials      1434     1434              

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Feb 18 '24 14:02 codecov[bot]

このPRによるapi.jsonの差分

差分はこちら

Get diff files from Workflow Page

github-actions[bot] avatar Feb 18 '24 14:02 github-actions[bot]

image

syuilo avatar Mar 02 '24 09:03 syuilo

image

syuilo avatar Mar 02 '24 09:03 syuilo

Allow edits and access to secrets by maintainersが無効になっている?

1Step621 avatar Mar 02 '24 09:03 1Step621

Allow edits and access to secrets by maintainersが無効になっているPRできないようにできないかしら

syuilo avatar Mar 02 '24 09:03 syuilo

単にカスタムCSSで指定しただけですが、Mac環境の参考までにどうぞ

Before

cyrillic-before

After

cyrillic-after

taiyme avatar Mar 06 '24 17:03 taiyme

うーん日本語文脈においては全角表示が正しいわけで, 言語設定依存で変えるべしな気がする

anatawa12 avatar Mar 06 '24 19:03 anatawa12

https://github.com/misskey-dev/misskey-hub-next/blob/14582eb05d2aea99740a567b8397c3b05d6d584b/public/fonts/fonts.css#L17-L42

みたいな指定が良さそう?

kakkokari-gtyih avatar Mar 20 '24 04:03 kakkokari-gtyih

うーん日本語文脈においては全角表示が正しいわけで

ほんまか?

tamaina avatar Mar 30 '24 13:03 tamaina

off-topic(そうでなくとも、言語設定依存で変えられるようになればCJKの漢字の重複してるところとかも言語別でよしなに指定できるようになってよさそうではある)

kakkokari-gtyih avatar Mar 30 '24 13:03 kakkokari-gtyih

https://github.com/misskey-dev/misskey-hub-next/blob/14582eb05d2aea99740a567b8397c3b05d6d584b/public/fonts/fonts.css#L17-L42

みたいな指定が良さそう?

良さそう

syuilo avatar Apr 02 '24 10:04 syuilo

これiOSで日本語を表示するとどうなるか知りたい

syuilo avatar Apr 02 '24 10:04 syuilo

off-topic(そうでなくとも、言語設定依存で変えられるようになればCJKの漢字の重複してるところとかも言語別でよしなに指定できるようになってよさそうではある)

一旦こっちで実装できるかやってみる(swのフォールバックはそういう機構をもたせると複雑化するからこの実装で問題ないと思われ)

kakkokari-gtyih avatar May 01 '24 04:05 kakkokari-gtyih