Merge branch 'fix-for-bem-modifiers' of https://github.com/nazar-pc/p…

…olymer into nazar-pc-fix-for-bem-modifiers
Polymer · Nov 4, 2015 · 7d905d5 · 7d905d5
2 parents 3378b4f + 35c89f1
commit 7d905d5
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 2 deletions.
diff --git a/src/lib/css-parse.html b/src/lib/css-parse.html
@@ -137,7 +137,7 @@
     },
 
     _hasMixinRules: function(rules) {
-      return (rules[0].selector.indexOf(this.VAR_START) >= 0);
+      return rules[0].selector.indexOf(this.VAR_START) === 0;
     },
 
     removeCustomProps: function(cssText) {

diff --git a/src/lib/style-properties.html b/src/lib/style-properties.html
@@ -166,7 +166,7 @@
               p || '';
           }
         }
-        return parts.join(';');
+        return parts.filter(function(v) {return v;}).join(';');
       },
 
       applyProperties: function(rule, props) {

diff --git a/test/unit/custom-style.html b/test/unit/custom-style.html
@@ -117,6 +117,18 @@
       border: 20px solid blue;
     }
   </style>
+  <style>
+    .foo--bar {
+      border-top : 3px solid red;
+    }
+  </style>
+  <style is="custom-style">
+    @media (min-width: 1px) {
+      .foo--bar {
+        border-top : 20px solid blue;
+      }
+    }
+  </style>
 </head>
 <body>
   <div class="italic">italic</div>
@@ -143,6 +155,7 @@
 
   <div class="foo"></div>
 
+  <div class="foo--bar style-scope"></div>
 
   <dom-module id="x-baz">
     <style>
@@ -461,6 +474,9 @@
         var computed = getComputedStyle(el);
         assert.equal(computed['font-family'].replace(/['"]+/g, ''), 'Varela font');
       });
+      test('BEM-like CSS selectors under media queries', function() {
+        assertComputed(document.querySelector('.foo--bar'), '3px');
+      });
 
     });