BEMHTML: #214 Inverse xhtml option default value to false

bem · Jun 30, 2016 · 97574f5 · 97574f5
1 parent caad627
commit 97574f5
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/lib/bemhtml/index.js b/lib/bemhtml/index.js
@@ -6,7 +6,7 @@ var BEMXJST = require('../bemxjst');
 function BEMHTML(options) {
   BEMXJST.apply(this, arguments);
 
-  var xhtml = typeof options.xhtml === 'undefined' ? true : options.xhtml;
+  var xhtml = typeof options.xhtml === 'undefined' ? false : options.xhtml;
   this._shortTagCloser = xhtml ? '/>' : '>';
 
   this._elemJsInstances = options.elemJsInstances;

diff --git a/test/bemhtml-test.js b/test/bemhtml-test.js
@@ -93,7 +93,7 @@ describe('BEMHTML engine tests', function() {
       );
     }, {
       block: 'page', elem: 'css', url: 'ohai'
-    }, '<link rel="stylesheet" href="ohai"/>');
+    }, '<link rel="stylesheet" href="ohai">');
   });
 
   it('should support `.xjstOptions()`', function() {
@@ -110,7 +110,7 @@ describe('BEMHTML engine tests', function() {
     it('should close short tags by default', function() {
       compile('')
         .apply({ tag: 'br' })
-        .should.equal('<br/>');
+        .should.equal('<br>');
     });
 
     it('should not close short tags with xhtml: false', function() {

diff --git a/test/runtime-applyctx-test.js b/test/runtime-applyctx-test.js
@@ -13,7 +13,7 @@ describe('Runtime applyCtx()', function() {
     },
     { block: 'b1' },
     '<div class="b1">' +
-    '<div class="b2">omg</div><br class="b3"/></div>');
+    '<div class="b2">omg</div><br class="b3"></div>');
   });
 
   it('should work with both context and changes', function() {