CSSのコードレビューではどの範囲まで評価してますか？

[devdyaya]

Issueのタイトル

CSSのコードレビューは設計部分から評価してますか？ / CSS Evolution

質問をしたいAMAブースを選択してください

• [ ] フロントエンドの課題を啓蒙する方法 by @yhassy
• [ ] アクセシビリティ by @fukumotoy @makojo
• [ ] Progressive Web Apps by @jxck
• [ ] パフォーマンス by CyberAgent @houbin217jz
• [x] CSS Evolution by @hiloki @cssradar
• [ ] React（仮） by @pirosikick

質問

• CSS（SASS、PostCSS、Stylus etc）の設計が望ましいかまでレビューで評価していますか？
  • 設計の軸を定めるためにCSS設計手法やガイドラインを取り入れると思いますが、実際のレビュー基準や、カオスを避ける知見などがあれば教えてください。
• そもそもCSS全体のレビューめっちゃ大変じゃないですか？
  • まるっとサービスやデザインを刷新する時、全コードに目を通しますか？
  • サービスリニューアル or 1からつくる時って、どういった頻度でレビューしますか？
• もし最近レビュワー視点で「これはやめてくれ」みたいなことが合ったら聞きたいです

[Yeshi]

回答

• 設計がずれてたらコード書けない。

• 緑の会社では設計をレビューするという仕事してた。 @studiomohawk (cssradar)

  • PJスタート時でどういう設計でやっていくのか、とヒアリング
  • 真ん中ぐらいになったらもう1度その時点でレビュー。
  • リリース直前、そろそろ燃えそうだなと言うタイミングでレビュー。

やめて欲しい系の話

• いきなりまるっと完成したものを全部見てくれ、というのは辛い。
  • 細かいプルリクでください。2万行とか無理。
• コメントの話。//todo, //mixin だけとか、いや、なにするの？

[devdyaya]

@Yeshi AMAでの回答のコメント化ありがとうございます！ 改めてイベントの開催お疲れ様でした！