search

phi-jp / meltline

melt + inline. css library
0 stars 3 forks source link

methodにmax, minのsize追加 #76

Closed suico-jp closed 3 years ago

suico-jp commented 3 years ago

命名

max-width-○ min-width-○ max-height-○ min-height-○

phi-jp commented 3 years ago

@suico-jp width と max-width とで必要な数値の種類ってだいぶ差があるから, メソッド分けといてもらっていい？ define-size-max 的な

わりと meltline.css のサイズを気にして数値絞ったり, 必要に応じて関数分けてたくさん数値必要なやつとちょっとしか必要ないやつとで分けたりしてるんよね.

phi-jp commented 3 years ago

@suico-jp 完璧b LGTM

suico-jp commented 3 years ago

@phi-jp

width と max-width とで必要な数値の種類ってだいぶ差があるから, メソッド分けといてもらっていい？ define-size-max 的な わりと meltline.css のサイズを気にして数値絞ったり, 必要に応じて関数分けてたくさん数値必要なやつとちょっとしか必要ないやつとで分けたりしてるんよね.

修正しました！ https://github.com/phi-jp/meltline/pull/76/commits/047b9a2b57f1c59fc5a28f6273462c228d2ca8f7