Merge pull request #190 from zrajnis/remove-focus-on-submission-fail

remove focus on submission failure

typescript-react/package.json

@@ -97,5 +97,5 @@
     "test:watch": "react-scripts test --env=jsdom"
   },
   "source": "src/index.ts",
-  "version": "1.9.5"
+  "version": "1.9.6"
 }

typescript-react/src/components/Form/Form.tsx

@@ -33,16 +33,11 @@ const genericError = 'Please correct all the required fields.';
 const scrollToFirstError = (errors: FormErrors<FormData, ErrorType> | undefined = {}) => {
   const fieldNames = deepKeys(errors, true);
 
-  // select library and datepicker don't have a name attribute on input, so the only way to target them is through data-qa-element-id
-  // data-qa-element-id has an unfortunate naming, and changing it now would be a breaking change, would be much prettier if it was data-element-id
-  // adding another data attribute would just duplicate stuff, as it would have the same value as data-qa-element-id
   for (const fieldName of fieldNames) {
-    const dataQaEl = `[data-qa-element-id="${fieldName}"]`;
-    const el = document.querySelectorAll(`[name="${fieldName}"], ${dataQaEl} input, input${dataQaEl}`)[0] as HTMLElement;
+    const el = document.querySelectorAll(`[name="${fieldName}"]`)[0] as HTMLElement;
 
     if (el) {
       el.scrollIntoView({ behavior: 'smooth', block: 'center' });
-      el.focus({ preventScroll: true });
       break;
     }
   }