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

システムのデフォルトフォントの指定の整理(KosugiMaruの完全削除を含む) #318

Conversation

Npepperlinux
Copy link
Contributor

@Npepperlinux Npepperlinux commented Sep 16, 2023

現在、WindowsではBIZ UDPゴシック(Noto Sans CJK JPを読み込んでいない、KosugiMaruは #281 待ちClose)、Androidでは指定なし (同じくKosugiMaruが #281 待ち) Closeの状態になっています。
国際化対応(#164)やgoogle_fontsの設定をするまでの間の設定として、Windows、AndroidでNoto Sans CJK JPを使用するように固定する提案です。

Close #489

@codecov-commenter
Copy link

codecov-commenter commented Sep 16, 2023

Codecov Report

Attention: Patch coverage is 56.41026% with 17 lines in your changes are missing coverage. Please review.

Project coverage is 36.60%. Comparing base (017c89e) to head (38cf5c6).

Files Patch % Lines
lib/view/themes/app_theme_scope.dart 56.41% 17 Missing ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #318      +/-   ##
===========================================
- Coverage    36.60%   36.60%   -0.01%     
===========================================
  Files          257      257              
  Lines        13587    13596       +9     
===========================================
+ Hits          4974     4977       +3     
- Misses        8613     8619       +6     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Npepperlinux Npepperlinux force-pushed the fonts_fix branch 3 times, most recently from 3972968 to fe24e07 Compare November 19, 2023 14:53
@Npepperlinux Npepperlinux changed the title Windows、AndroidのフォントをNoto Sans CJK JPに固定する システム標準時のSerifフォントの適用とKosugiMaru削除 Dec 25, 2023
@Npepperlinux
Copy link
Contributor Author

  • KosugiMaruが残っているのを解消しました
  • システムのSerifフォントが読み込まれていない(defaultFontNameが指定されていた)のを修正しました
    • OS毎にフォントを指定し直しましたが、カラー絵文字の指定が適切でない可能性があります

@poppingmoon
Copy link
Contributor

fontFamilyが指定されていなくてfontFamilyFallbackが指定されているときはfontFamilyFallbackの最初のフォントがfontFamilyとして扱われるみたいなのでfontFamilyFallbackだけ指定するようにすればもう少しシンプルになるんじゃないかと思います

https://api.flutter.dev/flutter/painting/TextStyle/fontFamilyFallback.html

@poppingmoon
Copy link
Contributor

dead codeがあります

vscodeであれば拡張機能を入れることでエラーや警告がわかりやすくなります
https://docs.flutter.dev/get-started/editor

@Npepperlinux Npepperlinux changed the title システム標準時のSerifフォントの適用とKosugiMaru削除 システムのデフォルトフォントの指定の整理(KosugiMaruの完全削除を含む) Jan 22, 2024
@Npepperlinux
Copy link
Contributor Author

CJKは限定的にNoto Sansの指定を言語ごとに切り替えたほうがいいかもしれない

@Npepperlinux Npepperlinux changed the base branch from develop to feature/miria_v2 October 21, 2024 06:42
@shiosyakeyakini-info shiosyakeyakini-info merged commit 303c695 into shiosyakeyakini-info:feature/miria_v2 Oct 21, 2024
1 check failed
@Npepperlinux Npepperlinux deleted the fonts_fix branch December 10, 2024 00:51
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.

要望:コードブロックはmonospaceフォントで表示して欲しい
4 participants