babel-plugin-jsx-dom-expressions 0.40.5 → 0.40.7

package/README.md

@@ -175,6 +175,15 @@ Removes tags from the template output if they have no closing parents and are th
 
 Removes quotes for html attributes when possible from the template output. This may not work in all browser-like environments the same. The solution has been tested again Chrome/Edge/Firefox/Safari.
 
+### omitAttributeSpacing
+
+- Type: `boolean`
+- Default: `true`
+
+When `true` (the default), the template output drops the whitespace between a quoted attribute and the next attribute (`<svg width="12"height="13">`). This mirrors the prior behavior and relies on the browser's HTML parser to tolerate the missing space. Some strict parsers (notably XML/SVG-only or game-engine browser-likes such as Coherent Gameface) reject the resulting markup.
+
+Set this to `false` to emit a single space between attributes (`<svg width="12" height="13">`) so the generated HTML is strictly valid. This may also be desirable together with `omitQuotes: false` / `omitLastClosingTag: false` when targeting stricter parsers.
+
 ### requireImportSource
 
 - Type: `string | false`

package/index.js

@@ -1477,7 +1477,7 @@ function transformAttributes$2(path, results) {
     }
 
     if (needsQuoting) {
-      needsSpacing = false;
+      needsSpacing = !config.omitAttributeSpacing;
       results.template += `="${escapeHTML(text, true)}"`;
     } else {
       needsSpacing = true;
@@ -2662,11 +2662,12 @@ function escapeExpression(path, expression, attr, escapeLiterals) {
     expression.alternate = escapeExpression(path, expression.alternate, attr, escapeLiterals);
     return expression;
   } else if (t__namespace.isLogicalExpression(expression)) {
-    expression.right = escapeExpression(path, expression.right, attr, escapeLiterals);
-    if (expression.operator !== "&&") {
-      expression.left = escapeExpression(path, expression.left, attr, escapeLiterals);
+    // Preserve the old short-circuit shape for && while still escaping the
+    // selected result of || and ?? as a whole.
+    if (expression.operator === "&&") {
+      expression.right = escapeExpression(path, expression.right, attr, escapeLiterals);
+      return expression;
     }
-    return expression;
   } else if (t__namespace.isCallExpression(expression) && t__namespace.isFunction(expression.callee)) {
     if (t__namespace.isBlockStatement(expression.callee.body)) {
       expression.callee.body.body = expression.callee.body.body.map((e) => {
@@ -2822,8 +2823,10 @@ function transformAttributes$1(path, results, info) {
         children = value;
       } else {
         let doEscape = true;
+
         if (key.startsWith("attr:")) key = key.replace("attr:", "");
-        if (BooleanAttributes.has(key)) {
+        if (BooleanAttributes.has(key) || key.startsWith("bool:")) {
+          key = key.replace("bool:", "");
           results.template.push("");
           const fn = t__namespace.callExpression(registerImportMethod(attribute, "ssrAttribute"), [
           t__namespace.stringLiteral(key),
@@ -4305,6 +4308,7 @@ var config = {
   omitNestedClosingTags: false,
   omitLastClosingTag: true,
   omitQuotes: true,
+  omitAttributeSpacing: true,
   contextToCustomElements: false,
   staticMarker: "@once",
   effectWrapper: "effect",

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "babel-plugin-jsx-dom-expressions",
   "description": "A JSX to DOM plugin that wraps expressions for fine grained change detection",
-  "version": "0.40.5",
+  "version": "0.40.7",
   "author": "Ryan Carniato",
   "license": "MIT",
   "repository": {
@@ -33,5 +33,5 @@
     "@rollup/plugin-babel": "6.0.3",
     "@types/babel__core": "^7.20.5"
   },
-  "gitHead": "d90e7885930e6bff605a1c50260577fda6506c23"
+  "gitHead": "3c01eb2ac61b193d4164319befe3f1b2634549cb"
 }