Skip to content

Commit 5b61286

Browse files
authored
Merge pull request #1139 from WonSong/dev
Move aria-label for carousel control buttons to the correct location
2 parents 53e54a1 + e208a6b commit 5b61286

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

src/controls/carousel/Carousel.tsx

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -89,8 +89,9 @@ export class Carousel extends React.Component<ICarouselProps, ICarouselState> {
8989
<div className={this.getMergedStyles(this.getButtonContainerStyles(), containerButtonsStyles)}
9090
onClick={() => { if (!prevButtonDisabled) { this.onCarouselButtonClicked(false); } }} >
9191
<IconButton
92+
ariaLabel={prevButtonAriaLabel}
9293
className={this.getMergedStyles(this.getButtonStyles(false), prevButtonStyles)}
93-
iconProps={{ iconName: prevButtonIconName, ariaLabel: prevButtonAriaLabel }}
94+
iconProps={{ iconName: prevButtonIconName }}
9495
disabled={prevButtonDisabled}
9596
onClick={() => { this.onCarouselButtonClicked(false); }} />
9697
</div>
@@ -116,8 +117,9 @@ export class Carousel extends React.Component<ICarouselProps, ICarouselState> {
116117
<div className={this.getMergedStyles(this.getButtonContainerStyles(), containerButtonsStyles)}
117118
onClick={() => { if (!nextButtonDisabled) { this.onCarouselButtonClicked(true); } }}>
118119
<IconButton
120+
ariaLabel={nextButtonAriaLabel}
119121
className={this.getMergedStyles(this.getButtonStyles(true), nextButtonStyles)}
120-
iconProps={{ iconName: nextButtonIconName, ariaLabel: nextButtonAriaLabel }}
122+
iconProps={{ iconName: nextButtonIconName }}
121123
disabled={nextButtonDisabled}
122124
onClick={() => { this.onCarouselButtonClicked(true); }} />
123125
</div>

0 commit comments

Comments
 (0)