fix: Remove image role attribute from Banner Icon component (#2663)

Co-authored-by: James Hancock <[email protected]>
trussworks · Jan 4, 2024 · 8ddf443 · 8ddf443
1 parent 6bdf5fb
commit 8ddf443
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 12 deletions.
diff --git a/src/components/banner/BannerIcon/BannerIcon.tsx b/src/components/banner/BannerIcon/BannerIcon.tsx
@@ -10,11 +10,9 @@ export const BannerIcon = ({
   const classes = classNames('usa-banner__icon usa-media-block__img', className)
 
   return (
-    // eslint-disable-next-line jsx-a11y/no-redundant-roles
     <img
       className={classes}
       src={src}
-      role="img"
       alt={alt}
       aria-hidden="true"
       {...imgProps}

diff --git a/src/components/banner/GovBanner/__snapshots__/GovBanner.test.tsx.snap b/src/components/banner/GovBanner/__snapshots__/GovBanner.test.tsx.snap
@@ -74,7 +74,6 @@ exports[`GovBanner component static content renders consistently in English for
             alt=""
             aria-hidden="true"
             className="usa-banner__icon usa-media-block__img"
-            role="img"
             src="test-file-stub"
           />
           <div
@@ -100,7 +99,6 @@ exports[`GovBanner component static content renders consistently in English for
             alt=""
             aria-hidden="true"
             className="usa-banner__icon usa-media-block__img"
-            role="img"
             src="test-file-stub"
           />
           <div
@@ -215,7 +213,6 @@ exports[`GovBanner component static content renders consistently in English for
             alt=""
             aria-hidden="true"
             className="usa-banner__icon usa-media-block__img"
-            role="img"
             src="test-file-stub"
           />
           <div
@@ -241,7 +238,6 @@ exports[`GovBanner component static content renders consistently in English for
             alt=""
             aria-hidden="true"
             className="usa-banner__icon usa-media-block__img"
-            role="img"
             src="test-file-stub"
           />
           <div
@@ -356,7 +352,6 @@ exports[`GovBanner component static content renders consistently in Spanish for
             alt=""
             aria-hidden="true"
             className="usa-banner__icon usa-media-block__img"
-            role="img"
             src="test-file-stub"
           />
           <div
@@ -382,7 +377,6 @@ exports[`GovBanner component static content renders consistently in Spanish for
             alt=""
             aria-hidden="true"
             className="usa-banner__icon usa-media-block__img"
-            role="img"
             src="test-file-stub"
           />
           <div
@@ -496,7 +490,6 @@ exports[`GovBanner component static content renders consistently in Spanish for
             alt=""
             aria-hidden="true"
             className="usa-banner__icon usa-media-block__img"
-            role="img"
             src="test-file-stub"
           />
           <div
@@ -522,7 +515,6 @@ exports[`GovBanner component static content renders consistently in Spanish for
             alt=""
             aria-hidden="true"
             className="usa-banner__icon usa-media-block__img"
-            role="img"
             src="test-file-stub"
           />
           <div
@@ -636,7 +628,6 @@ exports[`GovBanner component static content renders consistently with default pr
             alt=""
             aria-hidden="true"
             className="usa-banner__icon usa-media-block__img"
-            role="img"
             src="test-file-stub"
           />
           <div
@@ -662,7 +653,6 @@ exports[`GovBanner component static content renders consistently with default pr
             alt=""
             aria-hidden="true"
             className="usa-banner__icon usa-media-block__img"
-            role="img"
             src="test-file-stub"
           />
           <div