fix(action-sheet): allow scrollable action-sheet with many options (#…

…20145) fixes #17311
ionic-team · Jan 8, 2020 · 53fad97 · 53fad97
1 parent 3b4988a
commit 53fad97
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 3 deletions.
diff --git a/core/src/components/action-sheet/action-sheet.ios.scss b/core/src/components/action-sheet/action-sheet.ios.scss
@@ -32,8 +32,6 @@
 .action-sheet-group {
   @include border-radius($action-sheet-ios-border-radius);
   @include margin(null, null, $action-sheet-ios-group-margin-bottom - 2, null);
-
-  overflow: hidden;
 }
 
 .action-sheet-group:first-child {

diff --git a/core/src/components/action-sheet/action-sheet.scss b/core/src/components/action-sheet/action-sheet.scss
@@ -27,7 +27,7 @@
   --max-width: #{$action-sheet-max-width};
   --min-height: auto;
   --height: 100%;
-  --max-height: 100%;
+  --max-height: calc(100% - (var(--ion-safe-area-top) + var(--ion-safe-area-bottom)));
 
   @include font-smoothing();
   @include position(0, 0, 0, 0);