Skip to content

Commit 2b07f4d

Browse files
authored
Merge pull request #23 from midudev/feat/add-aria-label-and-role-to-arrow-navigations
Add role and aria-label to arrows
2 parents 0a74a24 + e5e0696 commit 2b07f4d

File tree

2 files changed

+5
-0
lines changed

2 files changed

+5
-0
lines changed

package.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,7 @@
4141
},
4242
"devDependencies": {
4343
"@babel/cli": "7",
44+
"@mdx-js/loader": "1.5.8",
4445
"@next/mdx": "9.3.4",
4546
"@s-ui/lint": "3",
4647
"@size-limit/preset-small-lib": "4.4.5",

src/react-slidy-slider.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -139,13 +139,17 @@ export default function ReactSlidySlider({
139139
{showArrows && (
140140
<>
141141
<span
142+
arial-label="Previous"
142143
className={`${classNameBase}-prev`}
144+
role="button"
143145
disabled={index === 0}
144146
onClick={handlePrev}
145147
/>
146148
<span
149+
arial-label="Next"
147150
className={`${classNameBase}-next`}
148151
disabled={items.length <= numOfSlides || index === items.length - 1}
152+
role="button"
149153
onClick={handleNext}
150154
/>
151155
</>

0 commit comments

Comments
 (0)