Skip to content

Commit

Permalink
carousel tweak
Browse files Browse the repository at this point in the history
  • Loading branch information
basher committed Apr 5, 2024
1 parent b75b456 commit 3d04199
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ ${style}
<section class="carousel-wrapper" aria-label="[meaningful label for carousel]"
>
<ul
class="carousel ${args.makeFullwidth === true ? 'carousel--fullwidth' : ''}"
class="carousel"
tabindex="0"
>
<li class="carousel__slide">
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ import * as WebUICarousel from './WebUICarousel.stories';
<Meta of={WebUICarousel} />

# `<webui-carousel>`
- This is an enhancement of the [default CSS carousel](/story/components-carousel--carousel) - showing a slide count, PREV and NEXT buttons, etc.
- This is an enhancement of the [default CSS carousel](/story/components-carousel--carousel) - optionally showing a slide count, PREV and NEXT buttons, etc.
- Native scrolling with `LEFT|RIGHT` arrow keys is overridden, allowing JavaScript to correctly update the slide count when using arrow keys.
- Each slide has an `aria-label`, indicating the current slide number and the total number of slides.
- When PREV and NEXT buttons are shown, keyboard `focus` is modified such that **only visible slides are focusable**.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@ export default {
},
},
argTypes: {
makeFullwidth: { control: 'boolean' },
showSlideCount: { control: 'boolean' },
showSlideCountPips: { control: 'boolean' },
showPrevNextButtons: { control: 'boolean' },
Expand Down

0 comments on commit 3d04199

Please sign in to comment.