DEV Community

Discussion on: How do you review CSS?

Collapse
 
wiommi profile image
William Iommi

Thank you all for your time/responses, much appreciate ❤️.

I'd say that our workflow is very similar to the one described by @robie577 and @antonfrattaroli.

We use BEM, Stylelint, SCSS variables, mixins, and so on. Could be complex when the project is big and you have to deal with many devs with different skills/seniority.

Anyway, I think that one of the important aspects is how solid is your UI / Design System, which is maybe close to what @ben said, if I understood correctly 😬.

If you have a poor UI/DS, you are open to a variety of ‘style combinations’ (even small stuff) that are ‘perfectly fine’ from a UI perspective, but they could lead you to a difficult review process with a bloated CSS code to maintain.