From 08d6f530163c004592701a683ee23b0e2c04c545 Mon Sep 17 00:00:00 2001
From: Felipe Simoes <felipe.simoes@netcentric.biz>
Date: Fri, 5 Jan 2024 15:52:00 +0100
Subject: [PATCH] Improve theme excel format

---
 blocks/theme/theme.js | 112 ++++++++++++++++++------------------------
 scripts/init.js       |   2 +-
 2 files changed, 48 insertions(+), 66 deletions(-)

diff --git a/blocks/theme/theme.js b/blocks/theme/theme.js
index 1892ddb9..c7ad257a 100644
--- a/blocks/theme/theme.js
+++ b/blocks/theme/theme.js
@@ -1,12 +1,15 @@
 import ComponentBase from '../../scripts/component-base.js';
 import { config } from '../../scripts/libs.js';
+// minify alias
+const k = Object.keys;
 
 export default class Theme extends ComponentBase {
   constructor() {
     super();
     this.external = '/theme.json';
-
-    this.applyToTag = ['font-size', 'font-weight', 'font-family'];
+    this.toTags = ['font-size', 'font-weight', 'font-family'];
+    this.fontFace = '';
+    this.atomic = '';
   }
 
   fontFaceTemplate(fontFace) {
@@ -17,36 +20,52 @@ export default class Theme extends ComponentBase {
       const lastBit = params.pop();
       const fontWeight = config.fontWeights[lastBit] || 'regular';
       const fontStyle = lastBit === 'italic' ? lastBit : 'normal';
-      return `
-@font-face {
-  font-family: ${name};
-  font-weight: ${fontWeight};
-  font-display: swap;
-  font-style: ${fontStyle};
-  src: url('/fonts/${fontFace}') format(${format});
-}
-`;
+      // eslint-disable-next-line max-len
+      return `\n@font-face {\nfont-family: ${name};\nfont-weight: ${fontWeight};\nfont-display: swap;\nfont-style: ${fontStyle};\nsrc: url('/fonts/${fontFace}') format(${format});\n}\n`;
     }
     return '';
   }
 
-  fontTagsTemplate(item, keys) {
-    return `${item['font-tag']} {${keys
-      .map((key) => {
-        if (this.headingVariables.includes(key) && item[key]) {
-          return `
-          ${key}: var(--scope-${key}, ${
-            item[key].indexOf(',') > -1 ? `'${item[key]}'` : item[key]
-          });`;
+  fontTags(t, index) {
+    const tag = t.tag[index];
+    const values = this.toTags.reduce((acc, key) => {
+      if (t[key][index]) {
+        if (acc[tag]) {
+          acc[tag][key] = t[key][index];
+        } else {
+          acc[tag] = { [key]: t[key][index] };
         }
-        return '';
-      })
-      .join('')}}\n`;
+      }
+      return acc;
+    }, {});
+    return k(values).map((value) => {
+      const val = values[value];
+      return `${tag} {\n${k(val)
+        .map((v) => `${v}: ${val[v]};`)
+        .join('\n')}\n}`;
+    });
+  }
+
+  renderVariables(key, row, t) {
+    const value = t[key][row];
+    let variable = '';
+    if (value) {
+      if (key === 'font-face') {
+        this.fontFace += this.fontFaceTemplate(value);
+      } else {
+        variable = `\n--raqn-${key}-${row}: ${value};\n`;
+        if (row === 'default') {
+          variable += `\n--scope-${key}: ${value};\n`;
+        }
+
+        this.atomic += `\n.${key}-${row} {\n--scope-${key}: var(--raqn-${key}-${row}, ${value}); \n}\n`;
+      }
+    }
+    return variable;
   }
 
-  createVariables() {
+  readValue() {
     const { data } = this.themeJson;
-    const k = Object.keys;
     const keys = data.map((item) => item.key);
     const t = data.reduce(
       (ac, item, i) =>
@@ -62,50 +81,13 @@ export default class Theme extends ComponentBase {
     );
 
     this.tags = k(t.tag)
-      .map((index) => {
-        const tag = t.tag[index];
-        const values = this.applyToTag.reduce((acc, key) => {
-          if (t[key][index]) {
-            if (acc[tag]) {
-              acc[tag][key] = t[key][index];
-            } else {
-              acc[tag] = { [key]: t[key][index] };
-            }
-          }
-          return acc;
-        }, {});
-        return k(values).map((value) => {
-          const val = values[value];
-          return `${tag} {\n${k(val)
-            .map((v) => `${v}: ${val[v]};`)
-            .join('\n')}\n}`;
-        });
-      })
+      .map((index) => this.fontTags(t, index))
       .join('\n\n');
-    this.fontFace = '';
-    this.atomic = '';
+
     this.variables = k(t)
       .map((key) => {
         const rows = k(t[key]);
-        return rows
-          .map((row) => {
-            const value = t[key][row];
-            let variable = '';
-            if (value) {
-              if (key === 'font-face') {
-                this.fontFace += this.fontFaceTemplate(value);
-              } else {
-                variable = `\n--raqn-${key}-${row}: ${value};\n`;
-                if (row === 'default') {
-                  variable += `\n--scope-${key}: ${value};\n`;
-                }
-
-                this.atomic += `\n.${key}-${row} {\n--scope-${key}: var(--raqn-${key}-${row}, ${value}); \n}\n`;
-              }
-            }
-            return variable;
-          })
-          .join('');
+        return rows.map((row) => this.renderVariables(key, row, t)).join('');
       })
       .join('');
   }
@@ -120,7 +102,7 @@ export default class Theme extends ComponentBase {
   async processExternal(response) {
     if (response.ok) {
       this.themeJson = await response.json();
-      this.createVariables();
+      this.readValue();
       this.styles();
     }
   }
diff --git a/scripts/init.js b/scripts/init.js
index d5f1c50f..ebc9b3d9 100644
--- a/scripts/init.js
+++ b/scripts/init.js
@@ -40,7 +40,7 @@ function lcpPriority() {
 }
 
 export async function init(node = document) {
-  let blocks = Array.from(node.querySelectorAll('[class]:not([class^=raqn]'));
+  let blocks = Array.from(node.querySelectorAll('[class]:not([class^=style]'));
 
   if (node === document) {
     const header = node.querySelector('header');