Closed kurudrive closed 9 months ago
@kurudrive theme.json でサイトタイトルのスタイルを変更できることを確認しました ありがとうございます
1点気になるところ
フォントスタイルや色などは theme.json
に追加指定できますが fontSize
は CSS
にかかれているのが優先されるようです
フォントサイズは変更して欲しくないからでしょうか?
フォントスタイルや色などは theme.json に追加指定できますが fontSize は CSS にかかれているのが優先されるようです フォントサイズは変更して欲しくないからでしょうか?
特に意味はありません。以前は指定できなかった or 知らなかっただけだと思います。
変更した方が良い箇所、問題点があればプルリクいただければと思います(・w・
@sysbird これは確認中になってますがもうマージしちゃっても良いですかね(・w・?
@kurudrive わー、すみません マージいたします フォントサイズが別件で提案しますね
チケットへのリンク / 変更の理由(元のissueがあればリンクを貼り付ければOK)
まぁ デザイン指定はできるだけ theme.json からにした方が良い...
どういう変更をしたか?
サイトタイトルタグへのデザイン指定をCSSファイルからtheme.jsonに移行
ソースコードについて
プログラムの変更の場合
テストを書かないのは普通ではありません。書けるテストは極力書くようにしてください。 書いていない場合は書かない理由を記載してください。
その他
変更内容について何を確認したか、どういう方法で確認をしたかなど
レビュワーの確認方法・確認する内容など
レビュワーに回す前の確認事項
レビュワー向け
確認して変更が反映されていない場合の確認事項