impl

Author: yeonjuan

packages/eslint-plugin/lib/rules/use-standard-html/check-content-model.js

@@ -192,7 +192,7 @@ function oneOrMore(model, context, state, node) {
     }
   }
 
-  if (count <= 0) {
+  if (count <= 0 && !model.contents.has("#text")) {
     context.report({
       node,
       messageId: MESSAGE_IDS.REQUIRED,
@@ -216,6 +216,10 @@ function optional(model, state) {
     state.contentModelIndex++;
     return CONTINUE;
   }
+  if (shouldIgnoreChild(child)) {
+    state.childIndex++;
+    return CONTINUE;
+  }
   const name = getNodeName(child);
   if (model.contents.has(name)) {
     state.childIndex++;

packages/eslint-plugin/tests/rules/use-standard-html.test.js

@@ -32,6 +32,12 @@ ruleTester.run("use-standard-html", rule, {
     {
       code: `<fieldset><legend></legend></fieldset>`,
     },
+    {
+      code: `<fieldset>
+        <!-- comment -->
+        <legend></legend>
+      </fieldset>`,
+    },
   ],
   invalid: [
     // required
@@ -65,15 +71,15 @@ ruleTester.run("use-standard-html", rule, {
       code: `<mark><div></div></mark>`,
       errors: [
         {
-          messageId: "required",
+          messageId: "notAllowed",
         },
       ],
     },
     {
       code: `<fieldset><base></base></fieldset>`,
       errors: [
         {
-          messageId: "required",
+          messageId: "notAllowed",
         },
       ],
     },