phi-jp / meltline

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

rounded-top-left系を追加 #47

Closed Iruru-jp closed 4 years ago

Iruru-jp commented 4 years ago

実装内容

懸念点

phi-jp commented 4 years ago

@Iruru-jp あっ値の書き方は前のままで良かった

↓これね

border-top-right-radius: 0px !important;

@nekokuloid の参考にってのはあくまでプロパティの追加の仕方の話.

Iruru-jp commented 4 years ago

@phi-jp

border-top-right-radius: 0px * 1px !important;

因みにここで* 1pxしているのですが消した方が良いですか?

phi-jp commented 4 years ago

@Iruru-jp いや * 1px は単位を px にするためにやってるやつだから消すと意味変わってくるね

Iruru-jp commented 4 years ago

@phi-jp

いや * 1px は単位を px にするためにやってるやつだから消すと意味変わってくるね

なるほど!やっとなんで付けているのか理解出来ました!mm

phi-jp commented 4 years ago

@Iruru-jp 今の状態でバッチリb LGTM

Iruru-jp commented 4 years ago

@phi-jp

自分も色々と勉強になったので他のjrがみてLGTM付いたらmergeします!

kinchan-jp commented 4 years ago

LGTM

hayato-jp commented 4 years ago

LGTMです