Skip to content

Commit

Permalink
Fixed side and floating buttons
Browse files Browse the repository at this point in the history
  • Loading branch information
tejas-raskar committed Oct 31, 2024
1 parent f0b0599 commit 8543f1f
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions packages/blade/src/components/Carousel/Carousel.web.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -534,7 +534,7 @@ const Carousel = ({
flexDirection="row"
height="100%"
>
{shouldShowPrevButton && shouldNavButtonsFloat ? (
{shouldShowPrevButton && shouldNavButtonsFloat && shouldShowControls ? (
<BaseBox zIndex={2} position="absolute" left="spacing.11">
<NavigationButton
type="previous"
Expand All @@ -543,7 +543,7 @@ const Carousel = ({
/>
</BaseBox>
) : null}
{isNavButtonsOnSide ? (
{isNavButtonsOnSide && shouldShowControls ? (
<NavigationButton
type="previous"
variant={navigationButtonVariant}
Expand All @@ -564,7 +564,7 @@ const Carousel = ({
>
{children}
</CarouselBody>
{shouldShowNextButton && shouldNavButtonsFloat ? (
{shouldShowNextButton && shouldNavButtonsFloat && shouldShowControls ? (
<BaseBox zIndex={2} position="absolute" right="spacing.11">
<NavigationButton
onClick={goToNextSlide}
Expand All @@ -573,7 +573,7 @@ const Carousel = ({
/>
</BaseBox>
) : null}
{isNavButtonsOnSide ? (
{isNavButtonsOnSide && shouldShowControls ? (
<NavigationButton
onClick={goToNextSlide}
type="next"
Expand Down

0 comments on commit 8543f1f

Please sign in to comment.