[WNMGDS-1693] Remove banner role from Dialog (#2062)

Remove banner role from Dialog.
CMSgov · Aug 29, 2022 · 3e5cdd0 · 3e5cdd0
1 parent f1b6864
commit 3e5cdd0
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 3 deletions.
diff --git a/packages/design-system/src/components/Dialog/Dialog.tsx b/packages/design-system/src/components/Dialog/Dialog.tsx
@@ -379,7 +379,7 @@ export const Dialog = (props: DialogProps) => {
       {...modalProps}
     >
       <div role="document">
-        <header className={headerClassNames} role="banner">
+        <header className={headerClassNames}>
           {
             // TODO: make heading required after removing title
             (title || heading) && (

diff --git a/packages/design-system/src/components/Dialog/__snapshots__/Dialog.test.jsx.snap b/packages/design-system/src/components/Dialog/__snapshots__/Dialog.test.jsx.snap
@@ -6,7 +6,6 @@ exports[`Dialog close button text and variation can be changed 1`] = `
 >
   <header
     class="ds-c-dialog__header"
-    role="banner"
   >
     <h1
       class="ds-h2"
@@ -54,7 +53,6 @@ exports[`Dialog renders with additional classNames and size 1`] = `
 >
   <header
     class="ds-c-dialog__header test-header"
-    role="banner"
   >
     <h1
       class="ds-h2"