add role region for pagelayout content

primer · Aug 5, 2024 · d08d28f · d08d28f
1 parent f441fed
commit d08d28f
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/packages/react/src/PageLayout/PageLayout.features.stories.tsx b/packages/react/src/PageLayout/PageLayout.features.stories.tsx
@@ -162,8 +162,7 @@ export const NestedScrollContainer: StoryFn = args => (
           <Placeholder label="Header" height={64} />
         </PageLayout.Header>
         <PageLayout.Content padding="normal" width="large">
-          {/* eslint-disable-next-line github/a11y-role-supports-aria-props */}
-          <Box sx={{display: 'grid', gap: 3}} tabIndex={0} aria-label="Page content">
+          <Box sx={{display: 'grid', gap: 3}} tabIndex={0} role="region" aria-label="Page content">
             {Array.from({length: args.numParagraphsInContent}).map((_, i) => (
               <Box key={i} as="p" sx={{margin: 0}}>
                 Lorem ipsum dolor sit amet, consectetur adipiscing elit. Nam at enim id lorem tempus egestas a non
@@ -335,8 +334,7 @@ export const ScrollContainerWithinPageLayoutPane: StoryFn = () => (
           </Box>
         </PageLayout.Pane>
         <PageLayout.Content padding="normal" width="large">
-          {/* eslint-disable-next-line github/a11y-role-supports-aria-props */}
-          <Box sx={{display: 'grid'}} tabIndex={0} aria-label="Page content">
+          <Box sx={{display: 'grid'}} tabIndex={0} role="region" aria-label="Page content">
             <Placeholder label="Page content" height={1600} />
           </Box>
         </PageLayout.Content>