Code Review

Here is what I look for when reviewing CSS:

  1. "Magic" Numbers
  2. Difference-To-Comp
  3. Scalability (e.g. "would this UI element still behave/look near-or-identical to our intent if e.g. the width of the containing element changed?
  4. Repeated CSS
  5. Tricky Incanting
  6. Repeated Variables
  7. Unnecessary Specificity
  8. Selector Misqualification

When I review CSS I check out the branch and play with the element in the DOM inspector.

Points of Discipline