Merge pull request #60 from mreinstein/master

make class inclusion check truthy to match snabbdom. fixes #59
snabbdom · Jun 30, 2021 · 4ab896c · 4ab896c
2 parents 2c2ee16 + 0b992ea
commit 4ab896c
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/modules/class.js b/modules/class.js
@@ -14,7 +14,7 @@ module.exports = function classModule (vnode, attributes) {
   existing = existing.length > 0 ? existing.split(' ') : []
 
   forOwn(classes, function (value, key) {
-    if (value === true) {
+    if (value) {
       _add.push(key)
     } else {
       _remove.push(key)

diff --git a/test/index.js b/test/index.js
@@ -153,6 +153,16 @@ test('Modules', function (t) {
   }, 'Github')
   t.equal(renderToString(vnode), '<a id="github" class="a b" href="http://github.com" target="_blank">Github</a>', 'attrs 2')
 
+  vnode = h('img', {
+    attrs: {
+      src: 'https://test.com'
+    },
+    class: {
+      selected: 'true'
+    }
+  })
+  t.equal(renderToString(vnode), '<img class="selected" src="https://test.com">', 'class truthy check')
+
   vnode = h('a#github', {
     attrs: {
       id: 'overridden',