Fix attribute selector parsing (#99)

rviscomi · web-flow · commit 8bee9797e738 · 2024-04-17T11:26:54.000-04:00
diff --git a/crx/capo.js b/crx/capo.js
diff --git a/crx/manifest.json b/crx/manifest.json
@@ -2,7 +2,7 @@
   "manifest_version": 3,
   "name": "Capo: get your ﹤𝚑𝚎𝚊𝚍﹥ in order",
   "description": "Visualize the optimal ordering of ﹤𝚑𝚎𝚊𝚍﹥ elements on any web page",
-  "version": "1.4.8",
+  "version": "1.4.9",
   "permissions": [
     "scripting",
     "activeTab",
diff --git a/docs/src/lib/capo.js b/docs/src/lib/capo.js
@@ -122,8 +122,14 @@ class $33f7359dc421be0c$export$8f8422ac5947a789 {
         const attributes = selector.match(/\[([A-Za-z-]+)="([^"]+)"\]/g) || [];
         // Set the attributes on the new element
         attributes.forEach((attribute)=>{
-            const [key, value] = attribute.replace("[", "").replace("]", "").split("=");
-            element.setAttribute(key, value.slice(1, -1));
+            // Trim square brackets
+            attribute = attribute.slice(1, -1);
+            const delimeterPosition = attribute.indexOf("=");
+            // Everything before the =
+            const key = attribute.slice(0, delimeterPosition);
+            // Everything after the =, with quotes trimmed
+            const value = attribute.slice(delimeterPosition + 1).slice(1, -1);
+            element.setAttribute(key, value);
         });
         return element;
     }
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@rviscomi/capo.js",
-  "version": "1.4.8",
+  "version": "1.4.9",
   "description": "Get your ﹤𝚑𝚎𝚊𝚍﹥ in order",
   "author": "Rick Viscomi",
   "license": "Apache-2.0",
diff --git a/snippet/capo.js b/snippet/capo.js
@@ -123,8 +123,14 @@ class $d410929ede0a2ee4$export$8f8422ac5947a789 {
         const attributes = selector.match(/\[([A-Za-z-]+)="([^"]+)"\]/g) || [];
         // Set the attributes on the new element
         attributes.forEach((attribute)=>{
-            const [key, value] = attribute.replace("[", "").replace("]", "").split("=");
-            element.setAttribute(key, value.slice(1, -1));
+            // Trim square brackets
+            attribute = attribute.slice(1, -1);
+            const delimeterPosition = attribute.indexOf("=");
+            // Everything before the =
+            const key = attribute.slice(0, delimeterPosition);
+            // Everything after the =, with quotes trimmed
+            const value = attribute.slice(delimeterPosition + 1).slice(1, -1);
+            element.setAttribute(key, value);
         });
         return element;
     }
diff --git a/src/lib/io.js b/src/lib/io.js
@@ -116,11 +116,14 @@ export class IO {
 
     // Set the attributes on the new element
     attributes.forEach((attribute) => {
-      const [key, value] = attribute
-        .replace("[", "")
-        .replace("]", "")
-        .split("=");
-      element.setAttribute(key, value.slice(1, -1));
+      // Trim square brackets
+      attribute = attribute.slice(1, -1);
+      const delimeterPosition = attribute.indexOf("=");
+      // Everything before the =
+      const key = attribute.slice(0, delimeterPosition);
+      // Everything after the =, with quotes trimmed
+      const value = attribute.slice(delimeterPosition + 1).slice(1, -1);
+      element.setAttribute(key, value);
     });
 
     return element;

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "@rviscomi/capo.js",`
`3`		`- "version": "1.4.8",`
	`3`	`+ "version": "1.4.9",`
`4`	`4`	`"description": "Get your ﹤𝚑𝚎𝚊𝚍﹥ in order",`
`5`	`5`	`"author": "Rick Viscomi",`
`6`	`6`	`"license": "Apache-2.0",`