@thecb/components 6.1.0-beta.7 → 6.1.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@thecb/components",
-  "version": "6.1.0-beta.7",
+  "version": "6.1.2",
   "description": "Common lib for CityBase react components",
   "main": "dist/index.cjs.js",
   "module": "dist/index.esm.js",

package/src/components/atoms/checkbox/Checkbox.js

@@ -124,6 +124,8 @@ const Checkbox = ({
             checked={checked}
             onChange={onChange}
             tabIndex="-1"
+            aria-invalid={error}
+            aria-describedby={`${name}-error-message`}
           />
           <StyledCheckbox
             error={error}

package/src/components/molecules/periscope-dashboard-iframe/PeriscopeDashboardIframe.js

@@ -48,10 +48,12 @@ const PeriscopeDashboardIframe = ({
 
   useEffect(() => {
     time.timer = setTimeout(() => {
-      periscopeDataRequestFailure();
+      if (!periscopeDataSuccess) {
+        periscopeDataRequestFailure();
+      }
     }, 10000);
     return () => clearTimeout(time.timer);
-  }, []);
+  }, [periscopeDataSuccess]);
 
   const Dashboard = height => url => (
     <DashboardIframe

package/src/util/focusFirstInvalidInputHook.js

@@ -1,17 +1,15 @@
 import { useEffect } from "react";
 
 const useFocusInvalidInput = (hasErrors, resetHasErrors = () => {}) => {
-  console.log("hey show errors", hasErrors);
   // Only move focus when "hasErrors" is true
   // "hasErrors" is managed by container page of form
   // typically set to "true" on attempted form submission, if errors exist
+  // Reset errors, if provided, resets the error state tracking in order to properly re-run
   useEffect(() => {
-    console.log("inside use effect show errors", hasErrors);
     if (hasErrors) {
       const inputsWithErrors = document.querySelectorAll(
         "input[aria-invalid=true]"
       );
-      console.log("inputs with errors is", inputsWithErrors);
       inputsWithErrors?.[0]?.focus();
     }
     return () => resetHasErrors(false);

package/src/util/general.js

@@ -46,19 +46,8 @@ export const safeChildren = (children, replacement = []) => {
   return unsafeValues.includes(children) ? replacement : children;
 };
 
-export const generateClickHandler = (
-  form,
-  handleErrors,
-  submitForm,
-  resetErrors
-) => e => {
+export const generateClickHandler = (form, handleErrors, submitForm) => e => {
   e.preventDefault();
-  if (resetErrors) {
-    console.log("reset errors", resetErrors);
-    // if provided, will reset error state tracker
-    // allows hooks that depend on error state to re-run
-    resetErrors(false);
-  }
   const formHasError = Object.values(form.fields).reduce(
     (acc, curr) => acc || curr.hasErrors,
     false