@yuneta/gobj-ui 7.7.0 → 7.7.1

package/dist/gobj-ui.cjs.js

@@ -7493,13 +7493,8 @@ function field_validation_message($input) {
 ************************************************************/
 function mark_field_validity($input) {
 	let $h = $input.parentNode ? $input.parentNode.querySelector(".help") : null;
-	let ok = true;
-	try {
-		ok = $input.checkValidity();
-	} catch (e) {
-		ok = true;
-	}
-	if (ok) {
+	let v = $input.validity;
+	if (!v || v.valid) {
 		$input.classList.remove("is-danger");
 		if ($h) {
 			$h.textContent = "";

package/dist/gobj-ui.es.js

@@ -7488,13 +7488,8 @@ function field_validation_message($input) {
 ************************************************************/
 function mark_field_validity($input) {
 	let $h = $input.parentNode ? $input.parentNode.querySelector(".help") : null;
-	let ok = true;
-	try {
-		ok = $input.checkValidity();
-	} catch (e) {
-		ok = true;
-	}
-	if (ok) {
+	let v = $input.validity;
+	if (!v || v.valid) {
 		$input.classList.remove("is-danger");
 		if ($h) {
 			$h.textContent = "";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yuneta/gobj-ui",
-  "version": "7.7.0",
+  "version": "7.7.1",
   "type": "module",
   "main": "dist/gobj-ui.cjs.js",
   "module": "dist/gobj-ui.es.js",

package/src/c_yui_form.js

@@ -1990,12 +1990,16 @@ function field_validation_message($input)
 function mark_field_validity($input)
 {
     let $h = $input.parentNode? $input.parentNode.querySelector('.help') : null;
-    let ok = true;
-    try {
-        ok = $input.checkValidity();
-    } catch(e) {
-        ok = true;      /*  not a validatable control  */
-    }
+
+    /*  `validity.valid`, NEVER `checkValidity()`.
+     *
+     *  checkValidity() FIRES the `invalid` event, and this function is what
+     *  that event's handler calls: one asks the other for ever. It cost a
+     *  page full of "too much recursion" the first time Save reported a bad
+     *  field. Reading `validity` asks the same question and fires nothing.
+     *  A control with no `validity` (a wrapper div) is not invalid.  */
+    let v = $input.validity;
+    let ok = !v || v.valid;
 
     if(ok) {
         $input.classList.remove('is-danger');