css-loader 0.28.0 → 0.28.4

package/CHANGELOG.md

@@ -2,6 +2,57 @@
 
 All notable changes to this project will be documented in this file. See [standard-version](https://github.com/conventional-changelog/standard-version) for commit guidelines.
 
+<a name="0.28.4"></a>
+## [0.28.4](https://github.com/webpack/css-loader/compare/v0.28.3...v0.28.4) (2017-05-30)
+
+
+### Bug Fixes
+
+* preserve leading underscore in class names ([#543](https://github.com/webpack/css-loader/issues/543)) ([f6673c8](https://github.com/webpack/css-loader/commit/f6673c8))
+
+
+
+<a name="0.28.3"></a>
+## [0.28.3](https://github.com/webpack/css-loader/compare/v0.28.2...v0.28.3) (2017-05-25)
+
+
+### Bug Fixes
+
+* correct plugin order for CSS Modules ([#534](https://github.com/webpack/css-loader/issues/534)) ([b90f492](https://github.com/webpack/css-loader/commit/b90f492))
+
+
+
+<a name="0.28.2"></a>
+## [0.28.2](https://github.com/webpack/css-loader/compare/v0.28.1...v0.28.2) (2017-05-22)
+
+
+### Bug Fixes
+
+* source maps path on `windows` ([#532](https://github.com/webpack/css-loader/issues/532)) ([c3d0d91](https://github.com/webpack/css-loader/commit/c3d0d91))
+
+
+
+<a name="0.28.1"></a>
+## [0.28.1](https://github.com/webpack/css-loader/compare/v0.28.0...v0.28.1) (2017-05-02)
+
+
+### Bug Fixes
+
+* allow to specify a full hostname as a root URL ([#521](https://github.com/webpack/css-loader/issues/521)) ([06d27a1](https://github.com/webpack/css-loader/commit/06d27a1))
+* case insensitivity of [@import](https://github.com/import) ([#514](https://github.com/webpack/css-loader/issues/514)) ([de4356b](https://github.com/webpack/css-loader/commit/de4356b))
+* don't handle empty [@import](https://github.com/import) and url() ([#513](https://github.com/webpack/css-loader/issues/513)) ([868fc94](https://github.com/webpack/css-loader/commit/868fc94))
+* imported variables are replaced in exports if followed by a comma ([#504](https://github.com/webpack/css-loader/issues/504)) ([956bad7](https://github.com/webpack/css-loader/commit/956bad7))
+* loader now correctly handles `url` with space(s) ([#495](https://github.com/webpack/css-loader/issues/495)) ([534ea55](https://github.com/webpack/css-loader/commit/534ea55))
+* url with a trailing space is now handled correctly ([#494](https://github.com/webpack/css-loader/issues/494)) ([e1ec4f2](https://github.com/webpack/css-loader/commit/e1ec4f2))
+* use `btoa` instead `Buffer` ([#501](https://github.com/webpack/css-loader/issues/501)) ([fbb0714](https://github.com/webpack/css-loader/commit/fbb0714))
+
+
+### Performance Improvements
+
+* generate source maps only when explicitly set ([#478](https://github.com/webpack/css-loader/issues/478)) ([b8f5c8f](https://github.com/webpack/css-loader/commit/b8f5c8f))
+
+
+
 <a name="0.28.0"></a>
 # [0.28.0](https://github.com/webpack/css-loader/compare/v0.27.3...v0.28.0) (2017-03-30)
 

package/README.md

@@ -6,7 +6,7 @@
 [![chat][chat]][chat-url]
 
 <div align="center">
-  <img width="200" height="200"
+  <img width="180" height="180" vspace="20"
     src="https://cdn.worldvectorlogo.com/logos/css-3.svg">
   <a href="https://github.com/webpack/webpack">
     <img width="200" height="200"
@@ -23,11 +23,16 @@ npm install --save-dev css-loader
 
 <h2 align="center">Usage</h2>
 
-The `css-loader` interprets `@import` and `url()` like `requires`.
+The `css-loader` interprets `@import` and `url()` like `import/require()`
+and will resolve them.
 
-Use the loader either via your webpack config, CLI or inline.
+Good loaders for requiring your assets are the [file-loader](https://github.com/webpack/file-loader)
+and the [url-loader](https://github.com/webpack/url-loader) which you should specify in your config (see [below](https://github.com/michael-ciniawsky/css-loader#assets)).
 
-### Via webpack config (recommended)
+**file.js**
+```js
+import css from 'file.css';
+```
 
 **webpack.config.js**
 ```js
@@ -43,109 +48,155 @@ module.exports = {
 }
 ```
 
-**In your application**
-```js
-import css from 'file.css';
-```
+### `toString`
 
-### CLI
+You can also use the css-loader results directly as string, such as in Angular's component style.
 
-```bash
-webpack --module-bind 'css=style-loader!css-loader'
+**webpack.config.js**
+```js
+{
+   test: /\.css$/,
+   use: [
+     'to-string-loader',
+     'css-loader'
+   ]
+}
 ```
 
-**In your application**
+or
+
 ```js
-import css from 'file.css';
+const css = require('./test.css').toString();
+
+console.log(css); // {String}
 ```
 
-### Inline
+If there are SourceMaps, they will also be included in the result string.
+
+If, for one reason or another, you need to extract CSS as a
+plain string resource (i.e. not wrapped in a JS module) you
+might want to check out the [extract-loader](https://github.com/peerigon/extract-loader).
+It's useful when you, for instance, need to post process the CSS as a string.
 
-**In your application**
+**webpack.config.js**
 ```js
-import css from 'style-loader!css-loader!./file.css';
+{
+   test: /\.css$/,
+   use: [
+     'handlebars-loader', // handlebars loader expects raw resource string
+     'extract-loader',
+     'css-loader'
+   ]
+}
 ```
 
 <h2 align="center">Options</h2>
 
-`@import` and `url()` are interpreted like `import` and will be resolved by the css-loader.
-Good loaders for requiring your assets are the [file-loader](https://github.com/webpack/file-loader)
-and the [url-loader](https://github.com/webpack/url-loader) which you should specify in your config (see below).
-
-To be compatible with existing css files (if not in CSS Module mode):
+|Name|Type|Default|Description|
+|:--:|:--:|:-----:|:----------|
+|**`root`**|`{String}`|`/`|Path to resolve URLs, URLs starting with `/` will not be translated|
+|**`url`**|`{Boolean}`|`true`| Enable/Disable `url()` handling|
+|**`alias`**|`{Object}`|`{}`|Create aliases to import certain modules more easily|
+|**`import`** |`{Boolean}`|`true`| Enable/Disable @import handling|
+|**`modules`**|`{Boolean}`|`false`|Enable/Disable CSS Modules|
+|**`minimize`**|`{Boolean\|Object}`|`false`|Enable/Disable minification|
+|**`sourceMap`**|`{Boolean}`|`false`|Enable/Disable Sourcemaps|
+|**`camelCase`**|`{Boolean\|String}`|`false`|Export Classnames in CamelCase|
+|**`importLoaders`**|`{Number}`|`0`|Number of loaders applied before CSS loader|
 
-* `url(image.png)` => `require('./image.png')`
-* `url(~module/image.png)` => `require('module/image.png')`
+### `root`
 
-<h2 align="center">Options</h2>
+For URLs that start with a `/`, the default behavior is to not translate them.
 
-|Name|Default|Description|
-|:--:|:-----:|:----------|
-|**`root`**|`/`|Path to resolve URLs, URLs starting with `/` will not be translated|
-|**`modules`**|`false`|Enable/Disable CSS Modules|
-|**`import`** |`true`| Enable/Disable @import handling|
-|**`url`**|`true`| Enable/Disable `url()` handling|
-|**`minimize`**|`false`|Enable/Disable minification|
-|**`sourceMap`**|`false`|Enable/Disable Sourcemaps|
-|**`camelCase`**|`false`|Export Classnames in CamelCase|
-|**`importLoaders`**|`0`|Number of loaders applied before CSS loader|
-|**`alias`**|`{}`|Create aliases to import certain modules more easily|
+`url(/image.png) => url(/image.png)`
 
-The following webpack config can load CSS files, embed small PNG/JPG/GIF/SVG images as well as fonts as [Data URLs](https://tools.ietf.org/html/rfc2397) and copy larger files to the output directory.
+If a `root` query parameter is set, however, it will be prepended to the URL
+and then translated.
 
 **webpack.config.js**
 ```js
-module.exports = {
-  module: {
-    rules: [
-      {
-        test: /\.css$/,
-        use: [ 'style-loader', 'css-loader' ]
-      },
-      {
-        test: /\.(png|jpg|gif|svg|eot|ttf|woff|woff2)$/,
-        loader: 'url-loader',
-        options: {
-          limit: 10000
-        }
-      }
-    ]
-  }
-};
+{
+  loader: 'css-loader',
+  options: { root: '.' }
+}
 ```
 
-### Root
+`url(/image.png)` => `require('./image.png')`
 
-For URLs that start with a `/`, the default behavior is to not translate them:
+Using 'Root-relative' urls is not recommended. You should only use it for legacy CSS files.
 
-* `url(/image.png)` => `url(/image.png)`
+### `url`
 
-If a `root` query parameter is set, however, it will be prepended to the URL
-and then translated:
+To disable `url()` resolving by `css-loader` set the option to `false`.
+
+To be compatible with existing css files (if not in CSS Module mode).
+
+```
+url(image.png) => require('./image.png')
+url(~module/image.png) => require('module/image.png')
+```
+
+### `alias`
+
+Rewrite your urls with alias, this is useful when it's hard to change url paths of your input files, for example, when you're using some css / sass files in another package (bootstrap, ratchet, font-awesome, etc.).
+
+`css-loader`'s `alias` follows the same syntax as webpack's `resolve.alias`, you can see the details at the [resolve docs] (https://webpack.js.org/configuration/resolve/#resolve-alias)
+
+**file.scss**
+```css
+@charset "UTF-8";
+@import "bootstrap";
+```
 
 **webpack.config.js**
 ```js
-rules: [
-  {
-    test: /\.css$/,
-    use: [
-      'style-loader',
-      {
-        loader: 'css-loader',
-        options: { root: '.' }
+{
+  test: /\.scss$/,
+  use: [
+    {
+      loader: "style-loader"
+    },
+    {
+      loader: "css-loader",
+      options: {
+        alias: {
+          "../fonts/bootstrap": "bootstrap-sass/assets/fonts/bootstrap"
+        }
       }
-    ]
-  }
-]
+    },
+    {
+      loader: "sass-loader",
+      options: {
+        includePaths: [
+          path.resolve("./node_modules/bootstrap-sass/assets/stylesheets")
+        ]
+      }
+    }
+  ]
+}
 ```
 
-* `url(/image.png)` => `require('./image.png')`
+Check out this [working bootstrap example](https://github.com/bbtfr/webpack2-bootstrap-sass-sample).
 
-Using 'Root-relative' urls is not recommended. You should only use it for legacy CSS files.
+### `import`
 
-### CSS Scope
+To disable `@import` resolving by `css-loader` set the option to `false`
+
+```css
+@import url('https://fonts.googleapis.com/css?family=Roboto');
+```
+
+> _⚠️ Use with caution, since this disables resolving for **all** `@import`s, including css modules `composes: xxx from 'path/to/file.css'` feature._
+
+### [`modules`](https://github.com/css-modules/css-modules)
+
+The query parameter `modules` enables the **CSS Modules** spec.
 
-By default CSS exports all class names into a global selector scope. Styles can be locally scoped to avoid globally scoping styles.
+This enables local scoped CSS by default. (You can switch it off with `:global(...)` or `:global` for selectors and/or rules.).
+
+#### `Scope`
+
+By default CSS exports all classnames into a global selector scope. Styles can be locally scoped to avoid globally scoping styles.
 
 The syntax `:local(.className)` can be used to declare `className` in the local scope. The local identifiers are exported by the module.
 
@@ -153,7 +204,6 @@ With `:local` (without brackets) local mode can be switched on for this selector
 
 The loader replaces local selectors with unique identifiers. The choosen unique identifiers are exported by the module.
 
-**app.css**
 ```css
 :local(.className) { background: red; }
 :local .className { color: green; }
@@ -161,17 +211,16 @@ The loader replaces local selectors with unique identifiers. The choosen unique
 :local .className .subClass :global(.global-class-name) { color: blue; }
 ```
 
-**app.bundle.css**
-``` css
+```css
 ._23_aKvs-b8bW2Vg3fwHozO { background: red; }
 ._23_aKvs-b8bW2Vg3fwHozO { color: green; }
 ._23_aKvs-b8bW2Vg3fwHozO ._13LGdX8RMStbBE9w-t0gZ1 { color: green; }
 ._23_aKvs-b8bW2Vg3fwHozO ._13LGdX8RMStbBE9w-t0gZ1 .global-class-name { color: blue; }
 ```
 
-> Note: Identifiers are exported
+> :information_source: Identifiers are exported
 
-``` js
+```js
 exports.locals = {
   className: '_23_aKvs-b8bW2Vg3fwHozO',
   subClass: '_13LGdX8RMStbBE9w-t0gZ1'
@@ -180,13 +229,14 @@ exports.locals = {
 
 CamelCase is recommended for local selectors. They are easier to use in the within the imported JS module.
 
-`url()` URLs in block scoped (`:local .abc`) rules behave like requests in modules:
+`url()` URLs in block scoped (`:local .abc`) rules behave like requests in modules.
 
-* `./file.png` instead of `file.png`
-* `module/file.png` instead of `~module/file.png`
+```
+file.png => ./file.png
+~module/file.png => module/file.png
+```
 
 You can use `:local(#someId)`, but this is not recommended. Use classes instead of ids.
-
 You can configure the generated ident with the `localIdentName` query parameter (default `[hash:base64]`).
 
  **webpack.config.js**
@@ -205,39 +255,29 @@ You can configure the generated ident with the `localIdentName` query parameter
 }
 ```
 
-You can also specify the absolute path to your custom `getLocalIdent` function to generate classname based on a different schema. Note that this requires `webpack >= v2.x.` since to be able to pass function in. For example:
+You can also specify the absolute path to your custom `getLocalIdent` function to generate classname based on a different schema. This requires `webpack >= 2.2.1` (it supports functions in the `options` object).
 
+**webpack.config.js**
 ```js
 {
-  test: /\.css$/,
-  use: [
-    {
-      loader: 'css-loader',
-      options: {
-        modules: true,
-        localIdentName: '[path][name]__[local]--[hash:base64:5]',
-        getLocalIdent: (context, localIdentName, localName, options) => {
-          return 'whatever_random_class_name'
-        }
-      }
+  loader: 'css-loader',
+  options: {
+    modules: true,
+    localIdentName: '[path][name]__[local]--[hash:base64:5]',
+    getLocalIdent: (context, localIdentName, localName, options) => {
+      return 'whatever_random_class_name'
     }
-  ]
+  }
 }
 ```
 
-Note: For prerendering with extract-text-webpack-plugin you should use `css-loader/locals` instead of `style-loader!css-loader` **in the prerendering bundle**. It doesn't embed CSS but only exports the identifier mappings.
-
-### [CSS Modules](https://github.com/css-modules/css-modules)
-
-The query parameter `modules` enables the **CSS Modules** spec.
-
-This enables local scoped CSS by default. (You can switch it off with `:global(...)` or `:global` for selectors and/or rules.)
+> :information_source: For prerendering with extract-text-webpack-plugin you should use `css-loader/locals` instead of `style-loader!css-loader` **in the prerendering bundle**. It doesn't embed CSS but only exports the identifier mappings.
 
-### CSS Composing
+#### `Composing`
 
-When declaring a local class name you can compose a local class from another local class name.
+When declaring a local classname you can compose a local class from another local classname.
 
-``` css
+```css
 :local(.className) {
   background: red;
   color: yellow;
@@ -249,7 +289,7 @@ When declaring a local class name you can compose a local class from another loc
 }
 ```
 
-This doesn't result in any change to the CSS itself but exports multiple class names:
+This doesn't result in any change to the CSS itself but exports multiple classnames.
 
 ```js
 exports.locals = {
@@ -269,18 +309,18 @@ exports.locals = {
 }
 ```
 
-### Importing CSS Locals
+#### `Importing`
 
-To import a local class name from another module:
+To import a local classname from another module.
 
-``` css
+```css
 :local(.continueButton) {
   composes: button from 'library/button.css';
   background: red;
 }
 ```
 
-``` css
+```css
 :local(.nameEdit) {
   composes: edit highlight from './edit.css';
   background: red;
@@ -289,7 +329,7 @@ To import a local class name from another module:
 
 To import from multiple modules use multiple `composes:` rules.
 
-``` css
+```css
 :local(.className) {
   composes: edit hightlight from './edit.css';
   composes: button from 'module/button.css';
@@ -298,177 +338,159 @@ To import from multiple modules use multiple `composes:` rules.
 }
 ```
 
-### SourceMaps
+### `minimize`
 
-To include Sourcemaps set the `sourceMap` query param.
+By default the css-loader minimizes the css if specified by the module system.
 
-I. e. the extract-text-webpack-plugin can handle them.
+In some cases the minification is destructive to the css, so you can provide your own options to the minifier if needed. cssnano is used for minification and you find a [list of options here](http://cssnano.co/options/).
 
-They are not enabled by default because they expose a runtime overhead and increase in bundle size (JS SourceMap do not). In addition to that relative paths are buggy and you need to use an absolute public path which include the server URL.
+You can also disable or enforce minification with the `minimize` query parameter.
 
 **webpack.config.js**
 ```js
 {
-  test: /\.css$/,
-  use: [
-    {
-      loader: 'css-loader',
-      options: {
-        sourceMap: true
-      }
-    }
-  ]
+  loader: 'css-loader',
+  options: {
+    minimize: true || {/* CSSNano Options */}
+  }
 }
 ```
 
-### toString
+### `sourceMap`
 
-You can also use the css-loader results directly as string, such as in Angular's component style.
+To include source maps set the `sourceMap` option.
 
-**webpack.config.js**
+I. e. the extract-text-webpack-plugin can handle them.
 
+They are not enabled by default because they expose a runtime overhead and increase in bundle size (JS source maps do not). In addition to that relative paths are buggy and you need to use an absolute public path which include the server URL.
+
+**webpack.config.js**
 ```js
 {
-   test: /\.css$/,
-   use: [
-     {
-       loaders: ['to-string-loader', 'css-loader']
-     }
-   ]
+  loader: 'css-loader',
+  options: {
+    sourceMap: true
+  }
 }
 ```
 
-or
+### `camelCase`
 
-```js
-const cssText = require('./test.css').toString();
-
-console.log(cssText);
-```
-
-If there are SourceMaps, they will also be included in the result string.
+By default, the exported JSON keys mirror the class names. If you want to camelize class names (useful in JS), pass the query parameter `camelCase` to css-loader.
 
-### ImportLoaders
+|Name|Type|Description|
+|:--:|:--:|:----------|
+|**`true`**|`{Boolean}`|Class names will be camelized|
+|**`'dashes'`**|`{String}`|Only dashes in class names will be camelized|
+|**`'only'`** |`{String}`|Introduced in `0.27.1`. Class names will be camelized, the original class name will be removed from the locals|
+|**`'dashesOnly'`**|`{String}`|Introduced in `0.27.1`. Dashes in class names will be camelized, the original class name will be removed from the locals|
 
-The query parameter `importLoaders` allow to configure which loaders should be applied to `@import`ed resources.
+**file.css**
+```css
+.class-name {}
+```
 
-`importLoaders`: That many loaders after the css-loader are used to import resources.
+**file.js**
+```js
+import { className } from 'file.css';
+```
 
 **webpack.config.js**
 ```js
 {
-  test: /\.css$/,
-  use: [
-    {
-      loader: 'css-loader',
-      options: {
-        importLoaders: 1
-      }
-    },
-    'postcss-loader'
-  ]
+  loader: 'css-loader',
+  options: {
+    camelCase: true
+  }
 }
 ```
 
-This may change in the future, when the module system (i. e. webpack) supports loader matching by origin.
-
-### Minification
+### `importLoaders`
 
-By default the css-loader minimizes the css if specified by the module system.
-
-In some cases the minification is destructive to the css, so you can provide some options to it. cssnano is used for minification and you find a [list of options here](http://cssnano.co/options/).
-
-You can also disable or enforce minification with the `minimize` query parameter.
+The query parameter `importLoaders` allows to configure how many loaders before `css-loader` should be applied to `@import`ed resources.
 
 **webpack.config.js**
 ```js
 {
   test: /\.css$/,
   use: [
+    'style-loader',
     {
       loader: 'css-loader',
       options: {
-        minimize: true || {/* CSSNano Options */}
+        importLoaders: 1 // 0 => no loaders (default); 1 => postcss-loader; 2 => postcss-loader, sass-loader
       }
-    }
+    },
+    'postcss-loader',
+    'sass-loader'
   ]
 }
 ```
 
-### CamelCase
+This may change in the future, when the module system (i. e. webpack) supports loader matching by origin.
 
-By default, the exported JSON keys mirror the class names. If you want to camelize class names (useful in JS), pass the query parameter `camelCase` to css-loader.
+<h2 align="center">Examples</h2>
 
-#### Possible Options
+### Assets
 
-|Option|Description|
-|:----:|:--------|
-|**`true`**|Class names will be camelized|
-|**`'dashes'`**|Only dashes in class names will be camelized|
-|**`'only'`** |Introduced in `0.27.1`. Class names will be camelized, the original class name will be removed from the locals|
-|**`'dashesOnly'`**|Introduced in `0.27.1`. Dashes in class names will be camelized, the original class name will be removed from the locals|
+The following `webpack.config.js` can load CSS files, embed small PNG/JPG/GIF/SVG images as well as fonts as [Data URLs](https://tools.ietf.org/html/rfc2397) and copy larger files to the output directory.
 
 **webpack.config.js**
 ```js
-{
-  test: /\.css$/,
-  use: [
-    {
-      loader: 'css-loader',
-      options: {
-        camelCase: true
+module.exports = {
+  module: {
+    rules: [
+      {
+        test: /\.css$/,
+        use: [ 'style-loader', 'css-loader' ]
+      },
+      {
+        test: /\.(png|jpg|gif|svg|eot|ttf|woff|woff2)$/,
+        loader: 'url-loader',
+        options: {
+          limit: 10000
+        }
       }
-    }
-  ]
+    ]
+  }
 }
 ```
 
-```css
-.class-name {}
-```
-
-```js
-import { className } from 'file.css';
-```
+### Extract
 
-### Alias
-
-Rewrite your urls with alias, this is useful when it's hard to change url paths of your input files, for example, when you're using some css / sass files in another package (bootstrap, ratchet, font-awesome, etc.).
-
-#### Possible Options
-
-css-loader's `alias` follows the same syntax as webpack's `resolve.alias`, you can see the details at: https://webpack.js.org/configuration/resolve/#resolve-alias
+For production builds it's recommended to extract the CSS from your bundle being able to use parallel loading of CSS/JS resources later on. This can be achieved by using the [extract-text-webpack-plugin](https://github.com/webpack-contrib/extract-text-webpack-plugin) to extract the CSS when running in production mode.
 
 **webpack.config.js**
 ```js
-{
-  test: /\.scss$/,
-  use: [{
-    loader: "style-loader"
-  }, {
-    loader: "css-loader",
-    options: {
-      alias: {
-        "../fonts/bootstrap": "bootstrap-sass/assets/fonts/bootstrap"
-      }
-    }
-  }, {
-    loader: "sass-loader",
-    options: {
-      includePaths: [
-        path.resolve("./node_modules/bootstrap-sass/assets/stylesheets")
+const env = process.env.NODE_ENV
+
+const ExtractTextPlugin = require('extract-text-webpack-plugin')
+
+module.exports = {
+  module: {
+    rules: [
+      {
+        test: /\.css$/,
+        use: env === 'production'
+          ? ExtractTextPlugin.extract({
+              fallback: 'style-loader',
+              use: [ 'css-loader' ]
+          })
+          : [ 'style-loader', 'css-loader' ]
+      },
+    ]
+  },
+  plugins: env === 'production'
+    ? [
+        new ExtractTextPlugin({
+          filename: '[name].css'
+        })
       ]
-    }
-  }]
+    : []
+  ]
 }
 ```
 
-```scss
-@charset "UTF-8";
-@import "bootstrap";
-```
-Check out this [working bootstrap example](https://github.com/bbtfr/webpack2-bootstrap-sass-sample).
-
 <h2 align="center">Maintainers</h2>
 
 <table>
@@ -476,29 +498,49 @@ Check out this [working bootstrap example](https://github.com/bbtfr/webpack2-boo
     <tr>
       <td align="center">
         <img width="150" height="150"
-        src="https://avatars3.githubusercontent.com/u/166921?v=3&s=150">
+        src="https://github.com/bebraw.png?v=3&s=150">
         </br>
         <a href="https://github.com/bebraw">Juho Vepsäläinen</a>
       </td>
       <td align="center">
         <img width="150" height="150"
-        src="https://avatars2.githubusercontent.com/u/8420490?v=3&s=150">
+        src="https://github.com/d3viant0ne.png?v=3&s=150">
         </br>
         <a href="https://github.com/d3viant0ne">Joshua Wiens</a>
       </td>
       <td align="center">
         <img width="150" height="150"
-        src="https://avatars3.githubusercontent.com/u/533616?v=3&s=150">
+        src="https://github.com/SpaceK33z.png?v=3&s=150">
         </br>
         <a href="https://github.com/SpaceK33z">Kees Kluskens</a>
       </td>
       <td align="center">
         <img width="150" height="150"
-        src="https://avatars3.githubusercontent.com/u/3408176?v=3&s=150">
+        src="https://github.com/TheLarkInn.png?v=3&s=150">
         </br>
         <a href="https://github.com/TheLarkInn">Sean Larkin</a>
       </td>
     </tr>
+    <tr>
+      <td align="center">
+        <img width="150" height="150"
+        src="https://github.com/michael-ciniawsky.png?v=3&s=150">
+        </br>
+        <a href="https://github.com/michael-ciniawsky">Michael Ciniawsky</a>
+      </td>
+      <td align="center">
+        <img width="150" height="150"
+        src="https://github.com/evilebottnawi.png?v=3&s=150">
+        </br>
+        <a href="https://github.com/evilebottnawi">Evilebot Tnawi</a>
+      </td>
+      <td align="center">
+        <img width="150" height="150"
+        src="https://github.com/joscha.png?v=3&s=150">
+        </br>
+        <a href="https://github.com/joscha">Joscha Feth</a>
+      </td>
+    </tr>
   <tbody>
 </table>
 

package/lib/css-base.js

@@ -54,7 +54,7 @@ function cssWithMappingToString(item, useSourceMap) {
 		return content;
 	}
 
-	if (useSourceMap) {
+	if (useSourceMap && typeof btoa === 'function') {
 		var sourceMapping = toComment(cssMapping);
 		var sourceURLs = cssMapping.sources.map(function (source) {
 			return '/*# sourceURL=' + cssMapping.sourceRoot + source + ' */'
@@ -68,8 +68,9 @@ function cssWithMappingToString(item, useSourceMap) {
 
 // Adapted from convert-source-map (MIT)
 function toComment(sourceMap) {
-  var base64 = new Buffer(JSON.stringify(sourceMap)).toString('base64');
-  var data = 'sourceMappingURL=data:application/json;charset=utf-8;base64,' + base64;
+	// eslint-disable-next-line no-undef
+	var base64 = btoa(unescape(encodeURIComponent(JSON.stringify(sourceMap))));
+	var data = 'sourceMappingURL=data:application/json;charset=utf-8;base64,' + base64;
 
-  return '/*# ' + data + ' */';
+	return '/*# ' + data + ' */';
 }

package/lib/loader.js

@@ -16,19 +16,35 @@ module.exports = function(content, map) {
 	var root = query.root;
 	var moduleMode = query.modules || query.module;
 	var camelCaseKeys = query.camelCase || query.camelcase;
+	var sourceMap = query.sourceMap || false;
 	var resolve = createResolver(query.alias);
 
-	if(map !== null && typeof map !== "string") {
-		map = JSON.stringify(map);
+	if(sourceMap) {
+		if (map) {
+			if (typeof map === "string") {
+				map = JSON.stringify(map);
+			}
+
+			if (map.sources) {
+				map.sources = map.sources.map(function (source) {
+					return source.replace(/\\/g, '/');
+				});
+				map.sourceRoot = '';
+			}
+		}
+	} else {
+		// Some loaders (example `"postcss-loader": "1.x.x"`) always generates source map, we should remove it
+		map = null;
 	}
 
 	processCss(content, map, {
 		mode: moduleMode ? "local" : "global",
-		from: loaderUtils.getRemainingRequest(this),
-		to: loaderUtils.getCurrentRequest(this),
+		from: loaderUtils.getRemainingRequest(this).split("!").pop(),
+		to: loaderUtils.getCurrentRequest(this).split("!").pop(),
 		query: query,
 		minimize: this.minimize,
-		loaderContext: this
+		loaderContext: this,
+		sourceMap: sourceMap
 	}, function(err, result) {
 		if(err) return callback(err);
 
@@ -98,11 +114,11 @@ module.exports = function(content, map) {
 			map = result.map;
 			if(map.sources) {
 				map.sources = map.sources.map(function(source) {
-					return source.split("!").pop();
+					return source.split("!").pop().replace(/\\/g, '/');
 				}, this);
 				map.sourceRoot = "";
 			}
-			map.file = map.file.split("!").pop();
+			map.file = map.file.split("!").pop().replace(/\\/g, '/');
 			map = JSON.stringify(map);
 			moduleJs = "exports.push([module.id, " + cssAsString + ", \"\", " + map + "]);";
 		} else {

package/lib/processCss.js

@@ -9,10 +9,12 @@ var loaderUtils = require("loader-utils");
 var assign = require("object-assign");
 var getLocalIdent = require("./getLocalIdent");
 
+var icssUtils = require('icss-utils');
 var localByDefault = require("postcss-modules-local-by-default");
 var extractImports = require("postcss-modules-extract-imports");
 var modulesScope = require("postcss-modules-scope");
 var modulesValues = require("postcss-modules-values");
+var valueParser = require('postcss-value-parser');
 
 var parserPlugin = postcss.plugin("css-loader-parser", function(options) {
 	return function(css) {
@@ -23,28 +25,34 @@ var parserPlugin = postcss.plugin("css-loader-parser", function(options) {
 
 		function replaceImportsInString(str) {
 			if(options.import) {
-				var tokens = str.split(/(\S+)/);
-				tokens = tokens.map(function (token) {
+				var tokens = valueParser(str);
+				tokens.walk(function (node) {
+					if (node.type !== 'word') {
+						return;
+					}
+					var token = node.value;
 					var importIndex = imports["$" + token];
 					if(typeof importIndex === "number") {
-						return "___CSS_LOADER_IMPORT___" + importIndex + "___";
+						node.value = "___CSS_LOADER_IMPORT___" + importIndex + "___";
 					}
-					return token;
-				});
-				return tokens.join("");
+				})
+				return tokens.toString();
 			}
 			return str;
 		}
 
 		if(options.import) {
-			css.walkAtRules("import", function(rule) {
+			css.walkAtRules(/import/i, function(rule) {
 				var values = Tokenizer.parseValues(rule.params);
 				var url = values.nodes[0].nodes[0];
 				if(url.type === "url") {
 					url = url.url;
 				} else if(url.type === "string") {
 					url = url.value;
-				} else throw rule.error("Unexpected format" + rule.params);
+				} else throw rule.error("Unexpected format " + rule.params);
+				if (!url.replace(/\s/g, '').length) {
+					return;
+				}
 				values.nodes[0].nodes.shift();
 				var mediaQuery = Tokenizer.stringifyValues(values);
 				if(loaderUtils.isUrlRequest(url, options.root) && options.mode === "global") {
@@ -58,24 +66,17 @@ var parserPlugin = postcss.plugin("css-loader-parser", function(options) {
 			});
 		}
 
-		css.walkRules(function(rule) {
-			if(rule.selector === ":export") {
-				rule.walkDecls(function(decl) {
-					exports[decl.prop] = decl.value;
-				});
-				rule.remove();
-			} else if(/^:import\(.+\)$/.test(rule.selector)) {
-				var match = /^:import\((.+)\)$/.exec(rule.selector);
-				var url = loaderUtils.parseString(match[1]);
-				rule.walkDecls(function(decl) {
-					imports["$" + decl.prop] = importItems.length;
-					importItems.push({
-						url: url,
-						export: decl.value
-					});
+		var icss = icssUtils.extractICSS(css);
+		exports = icss.icssExports;
+		Object.keys(icss.icssImports).forEach(function(key) {
+			var url = loaderUtils.parseString(key);
+			Object.keys(icss.icssImports[key]).forEach(function(prop) {
+				imports["$" + prop] = importItems.length;
+				importItems.push({
+					url: url,
+					export: icss.icssImports[key][prop]
 				});
-				rule.remove();
-			}
+			})
 		});
 
 		Object.keys(exports).forEach(function(exportName) {
@@ -97,8 +98,11 @@ var parserPlugin = postcss.plugin("css-loader-parser", function(options) {
 					}
 					break;
 				case "url":
-					if (options.url && !/^#/.test(item.url) && loaderUtils.isUrlRequest(item.url, options.root)) {
-						item.stringType = "";
+					if (options.url && item.url.replace(/\s/g, '').length && !/^#/.test(item.url) && loaderUtils.isUrlRequest(item.url, options.root)) {
+						// Don't remove quotes around url when contain space
+						if (item.url.indexOf(" ") === -1) {
+							item.stringType = "";
+						}
 						delete item.innerSpacingBefore;
 						delete item.innerSpacingAfter;
 						var url = item.url;
@@ -132,7 +136,7 @@ var parserPlugin = postcss.plugin("css-loader-parser", function(options) {
 
 module.exports = function processCss(inputSource, inputMap, options, callback) {
 	var query = options.query;
-	var root = query.root;
+	var root = query.root && query.root.length > 0 ? query.root.replace(/\/$/, "") : query.root;
 	var context = query.context;
 	var localIdentName = query.localIdentName || "[hash:base64]";
 	var localIdentRegExp = query.localIdentRegExp;
@@ -149,11 +153,14 @@ module.exports = function processCss(inputSource, inputMap, options, callback) {
 	};
 
 	var pipeline = postcss([
+		modulesValues,
 		localByDefault({
 			mode: options.mode,
 			rewriteUrl: function(global, url) {
 				if(parserOptions.url){
-					if(!loaderUtils.isUrlRequest(url, root)) {
+                    url = url.trim();
+
+					if(!url.replace(/\s/g, '').length || !loaderUtils.isUrlRequest(url, root)) {
 						return url;
 					}
 					if(global) {
@@ -164,7 +171,6 @@ module.exports = function processCss(inputSource, inputMap, options, callback) {
 			}
 		}),
 		extractImports(),
-		modulesValues,
 		modulesScope({
 			generateScopedName: function generateScopedName (exportName) {
 				return customGetLocalIdent(options.loaderContext, localIdentName, exportName, {
@@ -191,12 +197,12 @@ module.exports = function processCss(inputSource, inputMap, options, callback) {
 		// we need a prefix to avoid path rewriting of PostCSS
 		from: "/css-loader!" + options.from,
 		to: options.to,
-		map: {
+		map: options.sourceMap ? {
 			prev: inputMap,
 			sourcesContent: true,
 			inline: false,
 			annotation: false
-		}
+		} : null
 	}).then(function(result) {
 		callback(null, {
 			source: result.css,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "css-loader",
-  "version": "0.28.0",
+  "version": "0.28.4",
   "author": "Tobias Koppers @sokra",
   "description": "css loader module for webpack",
   "engines": {
@@ -15,6 +15,7 @@
     "babel-code-frame": "^6.11.0",
     "css-selector-tokenizer": "^0.7.0",
     "cssnano": ">=2.6.1 <4",
+    "icss-utils": "^2.1.0",
     "loader-utils": "^1.0.2",
     "lodash.camelcase": "^4.3.0",
     "object-assign": "^4.0.1",
@@ -23,6 +24,7 @@
     "postcss-modules-local-by-default": "^1.0.1",
     "postcss-modules-scope": "^1.0.0",
     "postcss-modules-values": "^1.1.0",
+    "postcss-value-parser": "^3.3.0",
     "source-list-map": "^0.1.7"
   },
   "devDependencies": {