Merge pull request #1050 from varya/fix/1037_active-modifier

Parse additional params correctly if many modifiers are in use. Fix #1037
SC5 · Dec 28, 2016 · a7b0fba · a7b0fba
2 parents 1afd060 + f97eb5e
commit a7b0fba
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 1 deletion.
diff --git a/lib/modules/kss-additional-params.js b/lib/modules/kss-additional-params.js
@@ -37,7 +37,7 @@ module.exports = {
     comment.split('\n\n').forEach(function(markUpBlock) {
 
       var varName = markUpBlock.match(/([^:^\n]*):[\s\S]*\n/);
-      if (varName && varName[1]) {
+      if (varName && varName[1] !== undefined) {
         varName = varName[1].trim();
       }
       if (varName && varName.substring(0, 3) === 'sg-') {

diff --git a/test/unit/modules/kss-additional-params.test.js b/test/unit/modules/kss-additional-params.test.js
@@ -16,6 +16,20 @@ describe('Parsing KSS additional params', () => {
     expect(params).eql(result);
   });
 
+  it('Should parse with KSS modifiers', () => {
+    // jscs:disable
+    var str = `default - Default button
+:hover - Hover button
+:active - Active button
+
+sg-param:
+Value`,
+    result = { 'sg-param': 'Value' },
+    value = kssAdditionalParams.get(str);
+    // jscs:enable
+    expect(value).eql(result);
+  });
+
   it('should parse windows linebreaks correctly', () => {
     // jscs:disable
     /*jshint -W109 */
@@ -219,4 +233,5 @@ describe('Parsing KSS additional params', () => {
     // jscs:enable
     expect(value).eql(result);
   });
+
 });