seperate concerns in auto-check-send into a new auto-check-loading event

koddsson · koddsson · commit b45dfcdd0163 · 2019-10-23T11:25:25.000+01:00
diff --git a/src/index.js b/src/index.js
@@ -99,9 +99,18 @@ function setLoadingState(event: Event) {
   const input = event.currentTarget
   if (!(input instanceof HTMLInputElement)) return
 
+  let message = 'Verifying…'
+  const setValidity = text => (message = text)
+  input.dispatchEvent(
+    new CustomEvent('auto-check-loading', {
+      bubbles: true,
+      detail: {setValidity}
+    })
+  )
+
   const autoCheckElement = input.closest('auto-check')
   if (autoCheckElement instanceof AutoCheckElement && autoCheckElement.required) {
-    input.setCustomValidity('Verifying…')
+    input.setCustomValidity(message)
   }
 }
 
@@ -152,12 +161,10 @@ async function check(autoCheckElement: AutoCheckElement) {
   if (id && id === state.previousValue) return
   state.previousValue = id
 
-  let message = 'Verifying…'
-  const setValidity = text => (message = text)
   input.dispatchEvent(
     new CustomEvent('auto-check-send', {
       bubbles: true,
-      detail: {body, setValidity}
+      detail: {body}
     })
   )
 
@@ -169,10 +176,6 @@ async function check(autoCheckElement: AutoCheckElement) {
     return
   }
 
-  if (autoCheckElement.required) {
-    input.setCustomValidity(message)
-  }
-
   if (state.controller) {
     state.controller.abort()
   } else {
