Skip to content

Commit 90aac85

Browse files
committed
chore(page): updated examples to provide a11y labels
1 parent 4ea82f7 commit 90aac85

File tree

6 files changed

+19
-18
lines changed

6 files changed

+19
-18
lines changed

packages/react-core/src/demos/examples/Page/PageContextSelector.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -325,13 +325,13 @@ export const PageStickySectionBreadcrumb: React.FunctionComponent = () => {
325325
mainContainerId={mainContainerId}
326326
isBreadcrumbWidthLimited
327327
>
328-
<PageSection isWidthLimited>
328+
<PageSection isWidthLimited aria-label="Main title">
329329
<Content>
330330
<Content component="h1">Main title</Content>
331331
<Content component="p">This is a full page demo.</Content>
332332
</Content>
333333
</PageSection>
334-
<PageSection isWidthLimited>
334+
<PageSection isWidthLimited aria-label="Card gallery">
335335
<Gallery hasGutter>
336336
{Array.from({ length: 50 }).map((_value, index) => (
337337
<GalleryItem key={index}>

packages/react-core/src/demos/examples/Page/PageStickySectionBreadcrumb.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -282,13 +282,13 @@ export const PageStickySectionBreadcrumb: React.FunctionComponent = () => {
282282
}
283283
}}
284284
>
285-
<PageSection isWidthLimited>
285+
<PageSection isWidthLimited aria-labelledby="main-title">
286286
<Content>
287-
<h1>Main title</h1>
287+
<h1 id="main-title">Main title</h1>
288288
<p>This is a full page demo.</p>
289289
</Content>
290290
</PageSection>
291-
<PageSection isWidthLimited>
291+
<PageSection isWidthLimited aria-label="Card gallery">
292292
<Gallery hasGutter>
293293
{Array.from({ length: 50 }).map((_value, index) => (
294294
<GalleryItem key={index}>

packages/react-core/src/demos/examples/Page/PageStickySectionGroup.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -278,9 +278,9 @@ export const PageStickySectionGroup: React.FunctionComponent = () => {
278278
isBreadcrumbWidthLimited
279279
isBreadcrumbGrouped
280280
additionalGroupedContent={
281-
<PageSection isWidthLimited>
281+
<PageSection isWidthLimited aria-labelledby="main-title">
282282
<Content>
283-
<h1>Main title</h1>
283+
<h1 id="main-title">Main title</h1>
284284
<p>This is a full page demo.</p>
285285
</Content>
286286
</PageSection>
@@ -289,7 +289,7 @@ export const PageStickySectionGroup: React.FunctionComponent = () => {
289289
stickyOnBreakpoint: { default: 'top' }
290290
}}
291291
>
292-
<PageSection>
292+
<PageSection aria-label="Card gallery">
293293
<Gallery hasGutter>
294294
{Array.from({ length: 50 }).map((_value, index) => (
295295
<GalleryItem key={index}>

packages/react-core/src/demos/examples/Page/PageStickySectionGroupAlternate.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -278,14 +278,14 @@ export const PageStickySectionGroupAlternate: React.FunctionComponent = () => {
278278
</BreadcrumbItem>
279279
</Breadcrumb>
280280
</PageBreadcrumb>
281-
<PageSection isWidthLimited>
281+
<PageSection isWidthLimited aria-labelledby="main-title">
282282
<Content>
283-
<h1>Main title</h1>
283+
<h1 id="main-title">Main title</h1>
284284
<p>This is a full page demo.</p>
285285
</Content>
286286
</PageSection>{' '}
287287
</PageGroup>
288-
<PageSection>
288+
<PageSection aria-label="Card gallery">
289289
<Gallery hasGutter>
290290
{Array.from({ length: 50 }).map((_value, index) => (
291291
<GalleryItem key={index}>

packages/react-core/src/demos/examples/PrimaryDetail/PrimaryDetailCardView.tsx

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -612,19 +612,19 @@ export const PrimaryDetailCardView: React.FunctionComponent = () => {
612612

613613
return (
614614
<DashboardWrapper mainContainerId="main-content-card-view-default-nav" breadcrumb={null}>
615-
<PageSection>
615+
<PageSection aria-labelledby="projects-heading">
616616
<Content>
617-
<h1>Projects</h1>
617+
<h1 id="projects-heading">Projects</h1>
618618
<p>This is a demo that showcases Patternfly cards.</p>
619619
</Content>
620620
</PageSection>
621-
<PageSection isFilled padding={{ default: 'noPadding' }}>
621+
<PageSection isFilled padding={{ default: 'noPadding' }} aria-label="Card filtering toolbar">
622622
<Toolbar id="card-view-data-toolbar-group-types" clearAllFilters={onDelete}>
623623
<ToolbarContent>{toolbarItems}</ToolbarContent>
624624
</Toolbar>
625625
<Divider component="div" />
626626
</PageSection>
627-
<PageSection isFilled padding={{ default: 'noPadding' }}>
627+
<PageSection isFilled padding={{ default: 'noPadding' }} aria-label="Card content area">
628628
<Drawer isExpanded={isDrawerExpanded} className={'pf-m-inline-on-2xl'}>
629629
<DrawerContent panelContent={panelContent}>
630630
<DrawerContentBody hasPadding>{drawerContent}</DrawerContentBody>
@@ -636,6 +636,7 @@ export const PrimaryDetailCardView: React.FunctionComponent = () => {
636636
stickyOnBreakpoint={{ default: 'bottom' }}
637637
padding={{ default: 'noPadding' }}
638638
variant="light"
639+
aria-label="Pagination controls"
639640
>
640641
<Pagination
641642
itemCount={filtered.length}

packages/react-core/src/demos/examples/PrimaryDetail/PrimaryDetailContentPadding.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -417,17 +417,17 @@ export const PrimaryDetailContentPadding: React.FunctionComponent = () => {
417417

418418
return (
419419
<DashboardWrapper>
420-
<PageSection>
420+
<PageSection aria-labelledby="main-title">
421421
<Content>
422-
<h1>Main title</h1>
422+
<h1 id="main-title">Main title</h1>
423423
<p>
424424
Body text should be Red Hat Text at 1rem(16px). It should have leading of 1.5rem(24px) because <br />
425425
of it's relative line height of 1.5.
426426
</p>
427427
</Content>
428428
</PageSection>
429429
<Divider component="div" />
430-
<PageSection padding={{ default: 'noPadding' }}>
430+
<PageSection padding={{ default: 'noPadding' }} aria-label="Drawer content section">
431431
<Drawer isExpanded={isDrawerExpanded}>
432432
<DrawerContent panelContent={panelContent} colorVariant="no-background">
433433
<DrawerContentBody hasPadding>{drawerContent}</DrawerContentBody>

0 commit comments

Comments
 (0)