Pin cssnano version that treats :focus-visible correctly (!14086) · Merge requests · Warren Gifford / sourcegraph

Warren Gifford requested to merge fix-focus-rings into main Sep 23, 2020

Created by: felixfbecker

Fixes https://github.com/sourcegraph/sourcegraph/issues/14085

Tracked the problem down to the CSS minifier (CSSNano) merging rules with the same selector in production builds. The :focus-visible PostCSS transform creates duplicate rules, but CSSNano combines them, which causes browsers that don't yet recognize :focus-visible to discard the entire rule. This was fixed in https://github.com/cssnano/cssnano/issues/642 but is not yet released on cssnano stable.

Pin cssnano version that treats :focus-visible correctly

Merge request reports