feat: unencosed content no longer requires commas w/ multiple attrs (#87

) (cherry picked from commit 7ca2017)
marko-js · Oct 12, 2021 · fd04465 · fd04465
1 parent 11b0fa8
commit fd04465
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 16 deletions.
diff --git a/src/states/OPEN_TAG.ts b/src/states/OPEN_TAG.ts
@@ -54,21 +54,6 @@ export const OPEN_TAG = Parser.createState({
           }
         }
       }
-
-      if (
-        tag.hasUnenclosedWhitespace &&
-        attributes.length > 1
-      ) {
-        for (let i = 0; i < attributes.length - 1; i++) {
-          if (!attributes[i].endedWithComma) {
-            this.notifyError(
-              attributes[i].pos,
-              "COMMAS_REQUIRED",
-              "commas are required to separate all attributes when using complex attribute values with un-enclosed whitespace"
-            );
-          }
-        }
-      }
     }
 
     tag.expectedCloseTagName = this.expectedCloseTagName =

diff --git a/test/autotest/attr-unenclosed-whitespace-multiple-attrs/expected.html b/test/autotest/attr-unenclosed-whitespace-multiple-attrs/expected.html
@@ -0,0 +1,5 @@
+<test a=(123 + 456) b=789>
+</test>
+text:"\n"
+<test a=123 b=(456 + 789)>
+</test>
diff --git a/...enclosed-whitespace-no-comma/input.htmljs → ...ed-whitespace-multiple-attrs/input.htmljs b/...enclosed-whitespace-no-comma/input.htmljs → ...ed-whitespace-multiple-attrs/input.htmljs
diff --git a/test/autotest/attr-unenclosed-whitespace-no-comma/expected.html b/test/autotest/attr-unenclosed-whitespace-no-comma/expected.html