Removing aria-live, updating test and changelog (cerner#3687)

saket2403 · Nov 21, 2022 · 65b9e71 · 65b9e71
1 parent c3d90ca
commit 65b9e71
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 6 deletions.
diff --git a/packages/terra-form-field/CHANGELOG.md b/packages/terra-form-field/CHANGELOG.md
@@ -7,6 +7,9 @@
   * Set the isLabelHidden prop to be private
   * Set an asterisk denoting required field to be aria-hidden
 
+* Removed
+  * Removed aria-live attribute from error message.
+
 ## 4.20.5 - (July 5, 2022)
 
 * Changed

diff --git a/packages/terra-form-field/src/Field.jsx b/packages/terra-form-field/src/Field.jsx
@@ -202,7 +202,7 @@ const Field = (props) => {
     <div style={customStyles} {...customProps} className={fieldClasses}>
       {labelGroup}
       {content}
-      {isInvalid && error && <div aria-live="assertive" id={htmlFor ? `${htmlFor}-error` : undefined} className={cx('error-text')}>{error}</div>}
+      {isInvalid && error && <div id={htmlFor ? `${htmlFor}-error` : undefined} className={cx('error-text')}>{error}</div>}
       {help && <div id={htmlFor ? `${htmlFor}-help` : undefined} className={cx('help-text')}>{help}</div>}
     </div>
   );

diff --git a/packages/terra-form-field/tests/jest/__snapshots__/Field.test.jsx.snap b/packages/terra-form-field/tests/jest/__snapshots__/Field.test.jsx.snap
@@ -271,7 +271,6 @@ exports[`should render a field in error 1`] = `
     </label>
   </div>
   <div
-    aria-live="assertive"
     className="error-text"
     id="test-error"
   >
@@ -668,7 +667,6 @@ exports[`should render a field with a hidden label in error 1`] = `
     </label>
   </div>
   <div
-    aria-live="assertive"
     className="error-text"
     id="test-error"
   >
@@ -827,7 +825,6 @@ exports[`should render a required field in error 1`] = `
     </label>
   </div>
   <div
-    aria-live="assertive"
     className="error-text"
     id="test-error"
   >
@@ -1266,7 +1263,6 @@ exports[`should render a required field with a hidden label in error 1`] = `
     </label>
   </div>
   <div
-    aria-live="assertive"
     className="error-text"
     id="test-error"
   >
@@ -2349,7 +2345,6 @@ exports[`should render an optional field in error 1`] = `
     </label>
   </div>
   <div
-    aria-live="assertive"
     className="error-text"
     id="test-error"
   >