Merge pull request #366 from eleven-labs/fix-class-nojs

Insert class="no-js" only if Modernizer is used
Swiip · Feb 19, 2015 · 2a53c8a · 2a53c8a
2 parents c7b3a0d + f3c3f6c
commit 2a53c8a
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/app/templates/src/_index.html b/app/templates/src/_index.html
@@ -1,5 +1,5 @@
 <!doctype html>
-<html class="no-js" ng-app="<%= appName %>">
+<html<% if (includeModernizr) { %> class="no-js"<% } %> ng-app="<%= appName %>">
   <head>
     <meta charset="utf-8">
     <title><%= appName %></title>

diff --git a/test/template/test-index-html.js b/test/template/test-index-html.js
@@ -62,10 +62,12 @@ describe('gulp-angular index js template', function () {
     model.includeModernizr = false;
     var result = indexHtml(model);
     result.should.not.match(/<!-- build:js\(src\) scripts\/modernizr.js -->/);
+    result.should.not.match(/<html class="no-js"/);
 
     model.includeModernizr = true;
     result = indexHtml(model);
     result.should.match(/<!-- build:js\(src\) scripts\/modernizr.js -->/);
+    result.should.match(/<html class="no-js"/);
   });
 
   it('should insert routerHtml content', function() {