babel-plugin-jsx-dom-expressions 0.36.9 → 0.36.18

package/README.md

@@ -130,7 +130,7 @@ Array of Component exports from module, that aren't included by default with the
 
 - Type: `string`
 - Default: `effect`
-  
+
 This plugin leverages a heuristic for reactive wrapping and lazy evaluation of JSX expressions. This option indicates the reactive wrapper function name (`effect`), defaults to `effect`.
 
 ### staticMarker
@@ -147,6 +147,19 @@ Comment decorator string indicates the static expression, used to tell the compi
 
 Memos let you efficiently use a derived value in many reactive computations. This option indicates the memo function name, defaults to `memo`.
 
+### validate
+
+- Type: `boolean`
+- Default: `true`
+
+Checks for properly formed HTML by checking for elements that would not be allowed in certain parent elements. This validation isn't complete but includes places where browser would "correct" it and break the DOM walks.
+
+### omitNestedClosingTags
+
+- Type: `boolean`
+- Default: `true`
+
+Removes unnecessary closing tags 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.
 
 ## Special Binding
 

package/index.js

@@ -707,7 +707,8 @@ const alwaysClose = [
   "select",
   "iframe",
   "script",
-  "template"
+  "template",
+  "fieldset"
 ];
 
 function transformElement$3(path, info) {
@@ -757,7 +758,9 @@ function transformElement$3(path, info) {
   results.template += ">";
   if (!voidTag) {
     // always close tags can still be skipped if they have no closing parents and are the last element
-    const toBeClosed = !info.lastElement || (info.toBeClosed && info.toBeClosed.has(tagName));
+    const toBeClosed =
+      !info.lastElement ||
+      (info.toBeClosed && (!config.omitNestedClosingTags || info.toBeClosed.has(tagName)));
     if (toBeClosed) {
       results.toBeClosed = new Set(info.toBeClosed || alwaysClose);
       results.toBeClosed.add(tagName);
@@ -3135,22 +3138,44 @@ function transformComponent(path) {
               checkTags: true
             })
           ) {
-            let expr =
+            if (
               config.wrapConditionals &&
               config.generate !== "ssr" &&
               (t__namespace.isLogicalExpression(value.expression) ||
                 t__namespace.isConditionalExpression(value.expression))
-                ? transformCondition(attribute.get("value").get("expression"), true)
-                : t__namespace.arrowFunctionExpression([], value.expression);
-            runningObject.push(
-              t__namespace.objectMethod(
-                "get",
-                id,
-                [],
-                t__namespace.blockStatement([t__namespace.returnStatement(expr.body)]),
-                !t__namespace.isValidIdentifier(key)
-              )
-            );
+            ) {
+              const expr = transformCondition(attribute.get("value").get("expression"), true);
+
+              runningObject.push(
+                t__namespace.objectMethod(
+                  "get",
+                  id,
+                  [],
+                  t__namespace.blockStatement([t__namespace.returnStatement(expr.body)]),
+                  !t__namespace.isValidIdentifier(key)
+                )
+              );
+            } else if (
+              t__namespace.isCallExpression(value.expression) &&
+              t__namespace.isArrowFunctionExpression(value.expression.callee)
+            ) {
+              const callee = value.expression.callee;
+              const body = t__namespace.isBlockStatement(callee.body)
+                ? callee.body
+                : t__namespace.blockStatement([t__namespace.returnStatement(callee.body)]);
+
+              runningObject.push(t__namespace.objectMethod("get", id, [], body, !t__namespace.isValidIdentifier(key)));
+            } else {
+              runningObject.push(
+                t__namespace.objectMethod(
+                  "get",
+                  id,
+                  [],
+                  t__namespace.blockStatement([t__namespace.returnStatement(value.expression)]),
+                  !t__namespace.isValidIdentifier(key)
+                )
+              );
+            }
           } else runningObject.push(t__namespace.objectProperty(id, value.expression));
         else runningObject.push(t__namespace.objectProperty(id, value));
       }
@@ -3454,6 +3479,7 @@ var config = {
   builtIns: [],
   requireImportSource: false,
   wrapConditionals: true,
+  omitNestedClosingTags: false,
   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.36.9",
+  "version": "0.36.18",
   "author": "Ryan Carniato",
   "license": "MIT",
   "repository": {
@@ -31,5 +31,5 @@
   "devDependencies": {
     "@babel/core": "^7.20.12"
   },
-  "gitHead": "eccf8327e8f09029d94273d4c9a70360651313e8"
+  "gitHead": "70ab5bf1b167c0a3cefd0a3582c0b97f502bb92b"
 }