Ignore: url(#...) (behavior property) (parcel-bundler#3887)

* Add test * Fix bug * Remove .only()
powerc9000 · Dec 9, 2019 · 9c883cb · 9c883cb
1 parent 7eab250
commit 9c883cb
Show file tree

Hide file tree

Showing 3 changed files with 22 additions and 1 deletion.
diff --git a/packages/core/integration-tests/test/css.js b/packages/core/integration-tests/test/css.js
@@ -241,6 +241,23 @@ describe('css', () => {
     );
   });
 
+  it('should ignore url() with IE behavior specifiers', async function() {
+    let b = await bundle(
+      path.join(__dirname, '/integration/css-url-behavior/index.css')
+    );
+
+    assertBundles(b, [
+      {
+        name: 'index.css',
+        assets: ['index.css']
+      }
+    ]);
+
+    let css = await outputFS.readFile(path.join(distDir, 'index.css'), 'utf8');
+
+    assert(css.includes('url(#default#VML)'));
+  });
+
   it('should minify CSS when minify is set', async function() {
     let b = await bundle(
       path.join(__dirname, '/integration/cssnano/index.js'),

diff --git a/packages/core/integration-tests/test/integration/css-url-behavior/index.css b/packages/core/integration-tests/test/integration/css-url-behavior/index.css
@@ -0,0 +1,3 @@
+.index {
+  behavior: url(#default#VML);
+}
diff --git a/packages/transformers/css/src/CSSTransformer.js b/packages/transformers/css/src/CSSTransformer.js
@@ -111,7 +111,8 @@ export default new Transformer({
           if (
             node.type === 'function' &&
             node.value === 'url' &&
-            node.nodes.length
+            node.nodes.length > 0 &&
+            !node.nodes[0].value.startsWith('#') // IE's `behavior: url(#default#VML)`
           ) {
             node.nodes[0].value = asset.addURLDependency(node.nodes[0].value, {
               loc: createDependencyLocation(