フロントエンドの個人的なレビュー観点
最近社内で新卒・初学者向けにフロントエンドのコードレビューをする機会が増えたので、自分のためにもレビューの観点をまとめておこうと思います。
共通
HTML
- ダグの使い方が適切か・意味のあるタグを使っているか
- 見出しやセクションが整っているか
- imgタグのalt属性が適切か
CSS
- marginとpaddingの適切な使い分け
- margin、paddingの方向(topやbottomなど)に統一感があるか
- id属性の重複チェック
- クラスに関して
- 崩れがないか・レスポンシブの場合どの幅でもきれいに見えるか
- (アクセシビリティチェック)
JavaScript
- 変数名、関数名の分かりやすさ、英単語
- if文、for文の乱用、多段ネスト、複雑な条件式
- DRY原則に則っているか
- エラーハンドリング
- console.log()等の消し忘れ
- JSでスタイルの変更を行う際に、JS側でスタイルの情報を持っていないか
- CSSで再現できる処理、アニメーション等がないかどうか
ブラウザチェック
- ターゲットブラウザできちんと動作するか
- 見た目に差が出ていないか
その他
- Lighthouseでのパフォーマンスチェック