Skip to content

Commit 226fc63

Browse files
Merge pull request #15 from topcoder-platform/qa-accessibility
Qa accessibility
2 parents 875f89d + 1884dc6 commit 226fc63

File tree

10 files changed

+24
-18
lines changed

10 files changed

+24
-18
lines changed

__tests__/shared/components/TopcoderHeader/__snapshots__/index.jsx.snap

+3-3
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,7 @@ exports[`Default render 1`] = `
7979
/>
8080
</span>
8181
<div
82-
aria-label="Search"
82+
aria-label="Find members by username or skill"
8383
className="src-shared-components-TopcoderHeader-___style__search-icon___3w5nA"
8484
data-menu="search"
8585
onBlur={[Function]}
@@ -372,7 +372,7 @@ exports[`Render with open menu 1`] = `
372372
/>
373373
</span>
374374
<div
375-
aria-label="Search"
375+
aria-label="Find members by username or skill"
376376
className="src-shared-components-TopcoderHeader-___style__search-icon___3w5nA"
377377
data-menu="search"
378378
onBlur={[Function]}
@@ -704,7 +704,7 @@ exports[`Render with specified profile 1`] = `
704704
</span>
705705
</div>
706706
<div
707-
aria-label="Search"
707+
aria-label="Find members by username or skill"
708708
className="opened src-shared-components-TopcoderHeader-___style__search-icon___3w5nA"
709709
data-menu="search"
710710
onBlur={[Function]}

__tests__/shared/components/__snapshots__/SortingSelectBar.jsx.snap

+4-4
Original file line numberDiff line numberDiff line change
@@ -4,11 +4,11 @@ exports[`Matches shallow shapshot 1`] = `
44
<div
55
className="src-shared-components-SortingSelectBar-___style__sortingBar___3esqf"
66
>
7-
<h1
7+
<h2
88
className="src-shared-components-SortingSelectBar-___style__title___36eG5"
99
>
1010
title
11-
</h1>
11+
</h2>
1212
<div
1313
className="src-shared-components-SortingSelectBar-___style__view-options-toggle-container___1N4rY"
1414
>
@@ -80,10 +80,10 @@ exports[`Matches shallow shapshot 2`] = `
8080
<div
8181
className="src-shared-components-SortingSelectBar-___style__sortingBar___3esqf"
8282
>
83-
<h1
83+
<h2
8484
className="src-shared-components-SortingSelectBar-___style__title___36eG5"
8585
>
8686
title
87-
</h1>
87+
</h2>
8888
</div>
8989
`;

__tests__/shared/components/challenge-listing/Filters/__snapshots__/ChallengeFilters.jsx.snap

+3-3
Original file line numberDiff line numberDiff line change
@@ -109,7 +109,7 @@ exports[`Matches shallow shapshot shapshot 2`] = `
109109
<span>
110110
<span
111111
aria-checked={false}
112-
aria-label="Filter"
112+
aria-label="Design toggle button pressed Off"
113113
className="src-shared-components-challenge-listing-Filters-___ChallengeFilters__filter-switch-with-label___21Ups"
114114
role="switch"
115115
>
@@ -126,7 +126,7 @@ exports[`Matches shallow shapshot shapshot 2`] = `
126126
</span>
127127
<span
128128
aria-checked={false}
129-
aria-label="Filter"
129+
aria-label="Development toggle button pressed Off"
130130
className="src-shared-components-challenge-listing-Filters-___ChallengeFilters__filter-switch-with-label___21Ups"
131131
role="switch"
132132
>
@@ -143,7 +143,7 @@ exports[`Matches shallow shapshot shapshot 2`] = `
143143
</span>
144144
<span
145145
aria-checked={false}
146-
aria-label="Filter"
146+
aria-label="Data Science toggle button pressed Off"
147147
className="src-shared-components-challenge-listing-Filters-___ChallengeFilters__filter-switch-with-label___21Ups"
148148
role="switch"
149149
>

__tests__/shared/components/challenge-listing/Filters/__snapshots__/ChallengeSearchBar.jsx.snap

+4
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,8 @@ exports[`Matches shallow shapshot 1`] = `
1616
className="src-shared-components-challenge-listing-Filters-ChallengeSearchBar-___style__SearchButton___3GzR0 src-shared-components-challenge-listing-Filters-ChallengeSearchBar-___style__active___8LjMb"
1717
onClick={[Function]}
1818
onKeyPress={[Function]}
19+
role="button"
20+
tabIndex="0"
1921
>
2022
<ZoomIcon
2123
className="src-shared-components-challenge-listing-Filters-ChallengeSearchBar-___style__zoomIcon___3d_cL"
@@ -44,6 +46,8 @@ exports[`Matches shallow shapshot 2`] = `
4446
className="src-shared-components-challenge-listing-Filters-ChallengeSearchBar-___style__SearchButton___3GzR0"
4547
onClick={[Function]}
4648
onKeyPress={[Function]}
49+
role="button"
50+
tabIndex="0"
4751
>
4852
<ZoomIcon
4953
className="src-shared-components-challenge-listing-Filters-ChallengeSearchBar-___style__zoomIcon___3d_cL"

src/shared/components/SortingSelectBar/index.jsx

+2-2
Original file line numberDiff line numberDiff line change
@@ -9,9 +9,9 @@ export default function SortingSelectBar({
99
}) {
1010
return (
1111
<div styleName="sortingBar">
12-
<h1 styleName="title">
12+
<h2 styleName="title">
1313
{title}
14-
</h1>
14+
</h2>
1515
{
1616
options ? (
1717
<div styleName="view-options-toggle-container">

src/shared/components/TopcoderHeader/index.jsx

+1-1
Original file line numberDiff line numberDiff line change
@@ -366,7 +366,7 @@ export default class TopcoderHeader extends React.Component {
366366
{userMenuHandle}
367367
{authButtons}
368368
<div
369-
aria-label="Search"
369+
aria-label="Find members by username or skill"
370370
role="button"
371371
tabIndex={0}
372372
data-menu="search"

src/shared/components/challenge-detail/Header/index.jsx

+1-1
Original file line numberDiff line numberDiff line change
@@ -262,7 +262,7 @@ export default function ChallengeHeader(props) {
262262
<div styleName="prizes-ops-container">
263263
<div styleName="prizes-outer-container">
264264
<h2 styleName="prizes-title">
265-
PRIZES
265+
Key Information
266266
</h2>
267267
<Prizes prizes={prizes && prizes.length ? prizes : [0]} pointPrizes={pointPrizes} />
268268
{

src/shared/components/challenge-listing/ChallengeCard/index.jsx

+1-1
Original file line numberDiff line numberDiff line change
@@ -131,7 +131,7 @@ function ChallengeCard({
131131
to={challengeDetailLink}
132132
styleName="challenge-title"
133133
openNewTab={openChallengesInNewTabs}
134-
><h3>{challenge.name}</h3>
134+
><p>{challenge.name}</p>
135135
</Link>
136136
<div styleName="details-footer">
137137
<span styleName="date">

src/shared/components/challenge-listing/Filters/ChallengeFilters.jsx

+3-3
Original file line numberDiff line numberDiff line change
@@ -76,21 +76,21 @@ export default function ChallengeFilters({
7676
isCardTypeSet === 'Challenges'
7777
? (
7878
<span>
79-
<span styleName="filter-switch-with-label" aria-label="Filter" role="switch" aria-checked={isTrackOn(TRACKS.DESIGN)}>
79+
<span styleName="filter-switch-with-label" aria-label={`Design toggle button pressed ${isTrackOn(TRACKS.DESIGN) ? 'On' : 'Off'}`} role="switch" aria-checked={isTrackOn(TRACKS.DESIGN)}>
8080
<SwitchWithLabel
8181
enabled={isTrackOn(TRACKS.DESIGN)}
8282
labelBefore="Design"
8383
onSwitch={on => switchTrack(TRACKS.DESIGN, on)}
8484
/>
8585
</span>
86-
<span styleName="filter-switch-with-label" aria-label="Filter" role="switch" aria-checked={isTrackOn(TRACKS.DEVELOP)}>
86+
<span styleName="filter-switch-with-label" aria-label={`Development toggle button pressed ${isTrackOn(TRACKS.DEVELOP) ? 'On' : 'Off'}`} role="switch" aria-checked={isTrackOn(TRACKS.DEVELOP)}>
8787
<SwitchWithLabel
8888
enabled={isTrackOn(TRACKS.DEVELOP)}
8989
labelBefore="Development"
9090
onSwitch={on => switchTrack(TRACKS.DEVELOP, on)}
9191
/>
9292
</span>
93-
<span styleName="filter-switch-with-label" aria-label="Filter" role="switch" aria-checked={isTrackOn(TRACKS.DATA_SCIENCE)}>
93+
<span styleName="filter-switch-with-label" aria-label={`Data Science toggle button pressed ${isTrackOn(TRACKS.DATA_SCIENCE) ? 'On' : 'Off'}`} role="switch" aria-checked={isTrackOn(TRACKS.DATA_SCIENCE)}>
9494
<SwitchWithLabel
9595
enabled={isTrackOn(TRACKS.DATA_SCIENCE)}
9696
labelBefore="Data Science"

src/shared/components/challenge-listing/Filters/ChallengeSearchBar/index.jsx

+2
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,8 @@ export default function ChallengeSearchBar({
3636
styleName={`SearchButton ${query ? 'active' : ''}`}
3737
onClick={() => onSearch(query.trim())}
3838
onKeyPress={() => onSearch(query.trim())}
39+
tabIndex="0"
40+
role="button"
3941
>
4042
<ZoomIcon styleName="zoomIcon" />
4143
</span>

0 commit comments

Comments
 (0)