Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ensure application CSS has priority over library CSS #362

Merged
merged 1 commit into from
Sep 8, 2020

Commits on Sep 8, 2020

  1. Ensure application CSS has priority over library CSS

    Import our application CSS as the very last thing, so that it can
    properly override  PatternFly variables. Before, our application CSS
    could land in the first third of dist/index.css *before* PatternFly's
    definitions, so that the latter overrode the former [1].
    
    This is a long-standing bug in mini-css-extract-plugin ([2] and
    countless things that point to it) with `NODE_ENV=production` builds.
    
    As a workaround, make sure that app.scss is the absolutely last imported
    CSS, instead of "almost last". It is still conceptually correct for the
    application CSS to be able to override patternfly-4-overrides.scss.
    
    [1] https://github.com/martinpitt/performance-graphs/issues/10
    [2] webpack-contrib/mini-css-extract-plugin#188
    martinpitt committed Sep 8, 2020
    Configuration menu
    Copy the full SHA
    fd9c45f View commit details
    Browse the repository at this point in the history