Allow arbitrary whitespace in CSS imports

Polymer · Aug 8, 2017 · 5c250d4 · 5c250d4
1 parent a7978e9
commit 5c250d4
Show file tree

Hide file tree

Showing 4 changed files with 18 additions and 2 deletions.
diff --git a/lib/utils/style-gather.html b/lib/utils/style-gather.html
@@ -46,7 +46,7 @@
      * @this {StyleGather}
      */
     cssFromModules(moduleIds) {
-      let modules = moduleIds.trim().split(' ');
+      let modules = moduleIds.trim().split(/\s+/);
       let cssText = '';
       for (let i=0; i < modules.length; i++) {
         cssText += this.cssFromModule(modules[i]);

diff --git a/test/unit/custom-style-import.html b/test/unit/custom-style-import.html
@@ -30,7 +30,8 @@
 <link rel="import" href="sub/style-import.html">
 
 <custom-style>
-<style is="custom-style" include="shared-style style-import">
+<style is="custom-style" include="shared-style style-import
+  style-import2">
   html {
 
     --import-mixin: {

diff --git a/test/unit/custom-style.html b/test/unit/custom-style.html
@@ -393,6 +393,11 @@
         assert.include(url, 'sub/google.png');
       });
 
+      test('style paths can have arbitrary whitespace', function() {
+        var foo = document.querySelector('.foo');
+        assertComputed(foo, '4px', 'width');
+      });
+
       test('imperative custom style', function() {
         var cs = document.createElement('custom-style');
         var style = document.createElement('style');

diff --git a/test/unit/sub/style-import.html b/test/unit/sub/style-import.html
@@ -8,4 +8,14 @@
       }
     </style>
   </template>
+</dom-module>
+
+<dom-module id="style-import2">
+  <template>
+    <style>
+      .foo {
+        width: 4px;
+      }
+    </style>
+  </template>
 </dom-module>