リセットCSSを導入する

syunto07ka commented 4 years ago

close #51

syunto07ka commented 4 years ago

効いてはいるみたいだけど、vscode上ではWarning

syunto07ka commented 4 years ago

@import-normalize is a non-standard at-rule. From the rule's documentation:

cssの基準的に違うのでlintをかませるとエラーになる。

エラーを解除する場合、 stylelintで引っかかっている場合は、stylelintconfigに記述を加える。 vscodeの機能として引っかかっている場合、「Unknown At Rules」をignoreにする。

syunto07ka commented 4 years ago

スクリーンショット 2020-04-26 19 24 27

エラーは消えた

syunto07ka commented 4 years ago

@piro0919 正解でしょうか？ (正直ルールを緩めたようにしか見えていないので正解わからず。。)

piro0919 commented 4 years ago

@syunto07ka

vscodeの機能として引っかかっている場合、「Unknown At Rules」をignoreにする。

こっちについては、調べた感じこれしかなさそうでしたただしワークスペース単位で設定して、git管理する必要はあるかなと

stylelintで引っかかっている場合は、stylelintconfigに記述を加える。

こっちについてはこれでも解決できますが、行単位で解決したほうがより好ましいかなーと思います Ignoring code

syunto07ka commented 4 years ago

stylelintの方については導入します(issueたてありがとうございます)

たしかに今全体設定としていますね。。そこの設定を加えてもう一度レビュー依頼しますー

syunto07ka commented 4 years ago

@piro0919 修正しましたので確認お願いします！

piro0919 commented 4 years ago

@syunto07ka

こちらが回答され次第レビューしますね 👍

syunto07ka commented 4 years ago

@piro0919 https://github.com/syunto07ka/jewelry_box/pull/64#issuecomment-619558510 の課題クリアできたと思うので、レビューお願いします！

syunto07ka / jewelry_box