如何在Pull Requests中审查CSS代码？

审查CSS代码与审查JavaScript代码有些不同。事实上，许多开发人员对CSS关注较少，这可能会导致问题。然而，许多开发人员不知道如何正确审查CSS代码。在本文中，我们将介绍在Pull Requests中审查CSS代码时应该注意的一些事项。

视觉检查

审查CSS的第一步是检查结果是否正确。这可能意味着将最终结果与设计模型进行比较，或者检查每个交互是否符合团队约定的动画原则。响应式也是一个重要因素，因此你应该检查页面在不同屏幕尺寸上的显示效果是否正确。视觉检查可能是最直接和重要的步骤，但不幸的是，许多开发人员就此停止。

代码风格

你的团队应该为CSS设置了一个代码检查器和格式化工具，如果还没有的话，你应该尽快设置。这将帮助你强制执行一致的代码风格，并使代码更易于阅读和维护。假设已经设置了代码检查器和格式化工具，你应该检查代码规范，这些规范无法由代码检查器自动强制执行。这些规范通常包括命名规范、适当的文档编写或使用CSS自定义属性代替硬编码的值。

特异性

CSS选择器很容易被滥用，导致以后出现问题。明确的规范通常可以解决许多问题，但有时会有一些问题被忽略。确保特异性尽可能低，选择器不要太通用或过于复杂，这将有助于提高代码的可维护性。

遗留代码

在理想情况下，Pull Request的作者对自己编写的CSS有清晰的视觉，一切都一次性完美地解决。但你知道，这种情况很少发生，意味着在开发过程中会进行实验和更改。随着代码的变化，一些旧代码可能会残留下来，对页面没有任何贡献。我经常遇到的一个常见例子是非flexbox元素的flexbox相关属性。这些需要一些时间来发现，但可以轻松删除，避免未来的问题。

性能

CSS性能经常被忽视。简单的规则、去重和最小化覆盖是一些可以提高性能的方法。可以理解的是，这种机会很难被发现，但是值得留意。话虽如此，不要过度优化性能。如果你不确定某个优化是否值得，那么你应该保持原样。