Merge pull request #86 from anttisalo/sectionMarkupFix

Fixed section.markup: false

Author: hannu

lib/app/views/partials/section.html

@@ -29,7 +29,7 @@ <h4 class="sg name">{{ mod.name | decodeHtml }}</h4>
       ng-if="!section.modifiers.length && section.markup"
       ng-bind-html="section.markup | setVariables: config.settings | unsafe" dynamic-compile>
     </div>
-    <div ng-if="section.markup && section.markup != 'false'">
+    <div ng-if="section.markup">
       <pre class="sg" ng-show="section.showMarkup">
         <span class="toggle hide-markup" ng-show="section.showMarkup" ng-click="hideSingleMarkupBox($index)">Hide markup</span>
         <code hljs source="section.markup" class="html"></code>

lib/styleguide.js

@@ -243,7 +243,7 @@ function jsonSections(sections) {
       deprecated: section.deprecated(),
       experimental: section.experimental(),
       reference: section.reference(),
-      markup: section.markup().toString()
+      markup: section.markup() ? section.markup().toString() : null
     }
   });
 }
@@ -255,7 +255,7 @@ function jsonModifiers(modifiers) {
       name: modifier.name(),
       description: sanitize(modifier.description()),
       className: modifier.className(),
-      markup: modifier.markup().toString()
+      markup: modifier.markup() ? modifier.markup().toString() : null
     }
   });
 }

test/structure.js

@@ -157,4 +157,12 @@ describe('styleguide.json', function() {
   it('should not reveal outputPath', function() {
     should.not.exist(jsonData.config.outputPath);
   });
+
+  it('should print markup if defined', function() {
+    jsonData.sections[1].markup.should.not.be.empty;
+  });
+
+  it('should not print empty markup', function() {
+    should.not.exist(jsonData.sections[2].markup);
+  });
 });