From 0a6c50b3bb70899fa3095d973b79085399f0d912 Mon Sep 17 00:00:00 2001
From: Bruno Windels <brunow@matrix.org>
Date: Fri, 14 Jun 2019 23:08:41 +0200
Subject: [PATCH] support bindings for className object

---
 src/ui/web/Template.js | 30 +++++++++++++++++++++---------
 1 file changed, 21 insertions(+), 9 deletions(-)

diff --git a/src/ui/web/Template.js b/src/ui/web/Template.js
index 5b275996..3bf08e03 100644
--- a/src/ui/web/Template.js
+++ b/src/ui/web/Template.js
@@ -1,5 +1,18 @@
 import { setAttribute, text, TAG_NAMES } from "./html.js";
 
+
+function classNames(obj, value) {
+    return Object.entries(obj).reduce((cn, [name, enabled]) => {
+        if (typeof enabled === "function") {
+            enabled = enabled(value);
+        }
+        if (enabled) {
+            return (cn.length ? " " : "") + name;
+        } else {
+            return cn;
+        }
+    }, "");
+}
 /*
     supports
         - event handlers (attribute fn value with name that starts with on)
@@ -20,11 +33,6 @@ export default class Template {
         this._attach();
     }
 
-    // TODO: obj needs to support bindings
-    classes(obj) {
-        Object.entries(obj).filter(([, value]) => value).map(([key]) => key).join(" ");
-    }
-
     root() {
         return this._root;
     }
@@ -98,6 +106,10 @@ export default class Template {
         binding();
     }
 
+    _addClassNamesBinding(node, obj) {
+        this._addAttributeBinding(node, "className", value => classNames(obj, value));
+    }
+
     _addTextBinding(fn) {
         const initialValue = fn(this._value);
         const node = text(initialValue);
@@ -130,7 +142,10 @@ export default class Template {
         if (attributes) {
             for(let [key, value] of Object.entries(attributes)) {
                 const isFn = typeof value === "function";
-                if (key.startsWith("on") && key.length > 2 && isFn) {
+                // binding for className as object of className => enabled
+                if (key === "className" && typeof value === "object" && value !== null) {
+                    this._addClassNamesBinding(node, value);
+                } else if (key.startsWith("on") && key.length > 2 && isFn) {
                     const eventName = key.substr(2, 1).toLowerCase() + key.substr(3);
                     const handler = value;
                     this._addEventListener(node, eventName, handler);
@@ -204,6 +219,3 @@ for (const tag of TAG_NAMES) {
         return this.el(tag, ... params);
     };
 }
-
-
-