Skip to content

Commit

Permalink
fix(component): Allow Carousels to fill height, resolves #168 (#208)
Browse files Browse the repository at this point in the history
Slightly tweak the default `Carousel` theme to make
slides fill the height specified in `base`, which
makes it much easier for users to override.
  • Loading branch information
tulup-conner authored Jun 8, 2022
1 parent 7ca9fcc commit 9ccf0c2
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/lib/theme/default.ts
Original file line number Diff line number Diff line change
Expand Up @@ -240,7 +240,7 @@ export default {
},
},
carousel: {
base: 'relative w-full',
base: 'relative h-full w-full min-h-56 sm:min-h-64 xl:min-h-80 2xl:min-h-96',
indicators: {
active: {
off: 'bg-white/50 hover:bg-white dark:bg-gray-800/50 dark:hover:bg-gray-800',
Expand All @@ -256,7 +256,7 @@ export default {
leftControl: 'absolute top-0 left-0 flex h-full items-center justify-center px-4 focus:outline-none',
rightControl: 'absolute top-0 right-0 flex h-full items-center justify-center px-4 focus:outline-none',
scrollContainer: {
base: 'flex h-56 snap-mandatory overflow-y-hidden overflow-x-scroll scroll-smooth rounded-lg sm:h-64 xl:h-80 2xl:h-96',
base: 'flex h-full snap-mandatory overflow-y-hidden overflow-x-scroll scroll-smooth rounded-lg',
snap: 'snap-x',
},
},
Expand Down

0 comments on commit 9ccf0c2

Please sign in to comment.