Merge pull request #1695 from Mahmoud-zino/bugfix/modal-top-scroll

bugfix: scroll to top of modal, when modal is bigger than view
skeletonlabs · Jul 19, 2023 · 9d07e49 · 9d07e49
2 parents 8efad37 + 1f00268
commit 9d07e49
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/.changeset/witty-pigs-wave.md b/.changeset/witty-pigs-wave.md
@@ -0,0 +1,5 @@
+---
+"@skeletonlabs/skeleton": patch
+---
+
+bugfix: fixed Scroll to the top of the modal, when the modal is bigger than the view.
diff --git a/packages/skeleton/src/lib/utilities/Modal/Modal.svelte b/packages/skeleton/src/lib/utilities/Modal/Modal.svelte
@@ -74,9 +74,9 @@
  export let regionFooter: CssClasses = 'flex justify-end space-x-2';
 
  // Base Styles
- const cBackdrop = 'fixed top-0 left-0 right-0 bottom-0';
- const cTransitionLayer = 'w-full h-full p-4 overflow-y-auto flex justify-center';
- const cModal = 'block'; // max-h-full overflow-y-auto overflow-x-hidden
+ const cBackdrop = 'fixed top-0 left-0 right-0 bottom-0 overflow-y-auto';
+ const cTransitionLayer = 'w-full h-fit min-h-full p-4 overflow-y-auto flex justify-center';
+ const cModal = 'block overflow-y-auto'; // max-h-full overflow-y-auto overflow-x-hidden
  const cModalImage = 'w-full h-auto';
 
  // Local