Skip to content

chore(spindle-ui): set base font family#8

Merged
herablog merged 1 commit into
masterfrom
chore/font-family
Aug 17, 2020
Merged

chore(spindle-ui): set base font family#8
herablog merged 1 commit into
masterfrom
chore/font-family

Conversation

@herablog

@herablog herablog commented Aug 6, 2020

Copy link
Copy Markdown
Member

Storybookのプレビュー内では、Spindleで規定されたフォントでコンポーネントを表示するようにします。

@herablog herablog requested a review from tokimari August 6, 2020 03:07
@herablog herablog self-assigned this Aug 6, 2020
@tokimari

tokimari commented Aug 6, 2020

Copy link
Copy Markdown
Contributor

baseで設定すべき内容なのでComponent単位で指定ではなく・・ですよね。
rem/emにする際のrootのfont sizeとかもどこで指定すべきかちょっと悩んでました:thinking:

@herablog

herablog commented Aug 6, 2020

Copy link
Copy Markdown
Member Author

そうそう、こりはあくまでStorybook内での指定という感じです!共通部分の実装が決まれば取り除いてもいいかもだね。

共通部分に関しては・・・

font-familyなどサイト間で違ってほしくないものは、まとめて共通CSS(ameba-color-palette.cssの上位版? or spindle.base.css or theme-ameba.css)作って読み込んでもらってもいいかなぁと思い中です。

rootのフォントサイズはサイトによって違ってもよさそうなのとすでに今違いそうなので、以下を検討してるけどどうだろかな。

  • コンポーネントではem指定しといて必要があれば親の要素で指定してもらう
  • カスタムプロパティで上書きできるようにしておく (ただしIE11サポートが厳しいかも, JS使えばなんとか・・)

@tokimari

Copy link
Copy Markdown
Contributor

spindle.base.css

よさそうです!
themeでもいいですが、全部がテーマみたいたものなので 💭

たしかにcomponentとしてはem指定のがよさそうですね。
親要素で指定するfont-sizeに応じて拡大縮小できればいいので、custom propertiesで上書きできなくても

コンポーネントではem指定しといて必要があれば親の要素で指定してもらう

だけでも十分そうな気もしますが、念のため・・??
componentの利用箇所によって、component内部の文字サイズの比率を変えられるのも、いかがなものなんだろうと。

@tokimari tokimari left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

書体指定はokです 🙆‍♀️
エンジニアを褒めるネコ:いいね

@herablog

Copy link
Copy Markdown
Member Author

フォントサイズに関しては、検討案はORでした(分かりづらく申し訳w)!ので、カスタムプロパティは必要に応じてにしましょう〜。

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