Skip to content

Commit

Permalink
fix: override style issues (close: #637) (#638)
Browse files Browse the repository at this point in the history
1. duplicated generated override style
2. unexpected style order, override style should be at the end of the extracted style bundle. (ref: webpack-contrib/mini-css-extract-plugin#130)
  • Loading branch information
shaun554 committed Jul 8, 2018
1 parent ebf8c1b commit 39a1b2b
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 3 deletions.
3 changes: 3 additions & 0 deletions lib/app/app.js
Expand Up @@ -7,6 +7,9 @@ import { siteData } from '@temp/siteData'
import enhanceApp from '@temp/enhanceApp'
import themeEnhanceApp from '@temp/themeEnhanceApp'

// generated from user config
import('@temp/override.styl')

// built-in components
import Content from './components/Content'
import OutboundLink from './components/OutboundLink.vue'
Expand Down
2 changes: 0 additions & 2 deletions lib/default-theme/styles/config.styl
Expand Up @@ -17,5 +17,3 @@ $MQMobileNarrow = 419px

// code
$lineNumbersWrapperWidth = 3.5rem

@import '~@temp/override.styl' // generated from user config
2 changes: 1 addition & 1 deletion lib/webpack/createBaseConfig.js
Expand Up @@ -49,7 +49,7 @@ module.exports = function createBaseConfig ({
: path.resolve(__dirname, './noopModule.js'))
.end()
.extensions
.merge(['.js', '.jsx', '.vue', '.json'])
.merge(['.js', '.jsx', '.vue', '.json', '.styl'])
.end()
.modules
// prioritize our own
Expand Down

0 comments on commit 39a1b2b

Please sign in to comment.