Skip to content

Commit 4c37fae

Browse files
committed
chore: update swiper and accessibillity
1 parent 31c8516 commit 4c37fae

File tree

4 files changed

+33
-18
lines changed

4 files changed

+33
-18
lines changed

packages/pluggableWidgets/carousel-web/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@
4444
"dependencies": {
4545
"@mendix/widget-plugin-component-kit": "workspace:*",
4646
"classnames": "^2.5.1",
47-
"swiper": "^11.2.10"
47+
"swiper": "^12.1.0"
4848
},
4949
"devDependencies": {
5050
"@mendix/automation-utils": "workspace:*",

packages/pluggableWidgets/carousel-web/src/components/Carousel.tsx

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import { ReactElement, ReactNode, useCallback, useState } from "react";
22
import { GUID } from "mendix";
33
import classNames from "classnames";
4-
import { A11y, Autoplay, EffectFade, Navigation, Pagination } from "swiper/modules";
4+
import { A11y, Autoplay, EffectFade, Keyboard, Navigation, Pagination } from "swiper/modules";
55
import { Swiper, SwiperClass, SwiperSlide } from "swiper/react";
66
import { PaginationOptions, SwiperOptions } from "swiper/types";
77
import "swiper/css";
@@ -57,10 +57,13 @@ export function Carousel(props: CarouselProps): ReactElement {
5757
effect: "fade",
5858
fadeEffect: { crossFade: true }
5959
}),
60-
modules: [A11y, Navigation, Pagination, EffectFade, Autoplay],
60+
modules: [A11y, Navigation, Pagination, EffectFade, Autoplay, Keyboard],
6161
a11y: {
6262
enabled: true,
6363
slideRole: "listitem"
64+
},
65+
keyboard: {
66+
enabled: true
6467
}
6568
};
6669

packages/pluggableWidgets/carousel-web/src/ui/Carousel.scss

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,12 @@
11
// Atlas UI compatible variable naming for data widgets
22
$gray-200: #e3e3e5 !default;
3-
$spacing-large: 24px !default;
3+
$spacing-small: 8px !default;
44
$focus-outline: 2px solid #264ae5 !default;
55
$focus-outline-offset: 2px !default;
66

77
.swiper {
8+
--swiper-navigation-size: 30px;
9+
--swiper-navigation-color: var(--brand-primary);
810
width: 100%;
911
height: 100%;
1012
}
@@ -31,16 +33,14 @@ $focus-outline-offset: 2px !default;
3133

3234
.swiper-button-next,
3335
.swiper-button-prev {
34-
--swiper-navigation-size: 24px;
3536
border-radius: 50%;
3637
background-color: var(--gray-200, $gray-200);
37-
padding: var(--spacing-large, $spacing-large);
38-
}
38+
padding: var(--spacing-small, $spacing-small);
3939

40-
.swiper-button-next:focus,
41-
.swiper-button-prev:focus {
42-
outline: var(--focus-outline, $focus-outline);
43-
outline-offset: var(--focus-outline-offset, $focus-outline-offset);
40+
&:focus {
41+
outline: var(--focus-outline, $focus-outline);
42+
outline-offset: var(--focus-outline-offset, $focus-outline-offset);
43+
}
4444
}
4545

4646
.swiper-slide img {

pnpm-lock.yaml

Lines changed: 19 additions & 7 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)