改修でdivタグを消して注意されたので戒めとしてメモする。(既存ソース改修)
やりたかったこと
<div class="hoge”>
のクラス名が悪さをしていたので消したかった。
やったこと
<div class="hoge”>
を削除。
その時考えていたこと
見た感じこのdivタグで何かやってる感じしないし、divタグごと消せばよくね
要らない物を残す必要ないし
結果
挙動的には問題ないのでプルリクを出す。
が、ソースレビューでNG
ダメだったところ
divタグとCSSのセレクタのことを詳しく知らずに安易に消した点
隣接セレクタの存在
隣接セレクタの存在とは
隣接セレクタとは同じ階層にある要素同士で、 ある要素の直後に隣接している要素を対象にスタイルを適用することができる存在
このdiv他の部分に影響を与える可能性があるということ。
まとめ
CSSにも注意して修正