Skip to content

Commit 8a57e49

Browse files
author
Piotr Siatka
committed
#227 Fix styling bugs.
Fix buttonContainer onClick logic.
1 parent d87d9bb commit 8a57e49

File tree

2 files changed

+5
-4
lines changed

2 files changed

+5
-4
lines changed

src/controls/carousel/Carousel.module.scss

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,7 @@
5151
.buttonsContainer {
5252
display: flex;
5353
background-color: transparent;
54+
cursor: pointer;
5455
}
5556
.buttonsOnlyContainer {
5657
@extend .buttonsContainer;
@@ -60,7 +61,7 @@
6061

6162
.blockButtonsContainer {
6263
@extend .buttonsContainer;
63-
height: 100%;
64+
min-height: 100%;
6465
min-width: 32px;
6566
}
6667
.blockButtonsContainer:hover {
@@ -71,7 +72,7 @@
7172

7273
.hiddenButtonsContainer {
7374
@extend .buttonsContainer;
74-
height: 100%;
75+
min-height: 100%;
7576
min-width: 32px;
7677
opacity: 0;
7778
}

src/controls/carousel/Carousel.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ export class Carousel extends React.Component<ICarouselProps, ICarouselState> {
6060

6161
return (
6262
<div className={this.getMergedStyles(styles.container, containerStyles)}>
63-
<div className={this.getMergedStyles(this.getButtonContainerStyles(), containerButtonsStyles)} onClick={() => { if (prevButtonDisabled) { this.onCarouselButtonClicked(false); } }} >
63+
<div className={this.getMergedStyles(this.getButtonContainerStyles(), containerButtonsStyles)} onClick={() => { if (!prevButtonDisabled) { this.onCarouselButtonClicked(false); } }} >
6464
<IconButton
6565
className={this.getMergedStyles(this.getButtonStyles(false), prevButtonStyles)}
6666
iconProps={{ iconName: prevButtonIconName }}
@@ -87,7 +87,7 @@ export class Carousel extends React.Component<ICarouselProps, ICarouselState> {
8787
}
8888
</div>
8989

90-
<div className={this.getMergedStyles(this.getButtonContainerStyles(), containerButtonsStyles)} onClick={() => { if (nextButtonDisabled) { this.onCarouselButtonClicked(true); } }}>
90+
<div className={this.getMergedStyles(this.getButtonContainerStyles(), containerButtonsStyles)} onClick={() => { if (!nextButtonDisabled) { this.onCarouselButtonClicked(true); } }}>
9191
<IconButton
9292
className={this.getMergedStyles(this.getButtonStyles(true), nextButtonStyles)}
9393
iconProps={{ iconName: nextButtonIconName }}

0 commit comments

Comments
 (0)