Fix scrollbar padding for safari breaks other component styling

Signed-off-by: Ajay Bura <ajbura@gmail.com>

diff --git a/src/app/atoms/scroll/ScrollView.scss b/src/app/atoms/scroll/ScrollView.scss
index 6c7d709fd7096a21ace6f3a6c921f809b715d22d..251037e18482ff45165227b7c84be417116b7981 100644 (file)
--- a/src/app/atoms/scroll/ScrollView.scss
+++ b/src/app/atoms/scroll/ScrollView.scss
@@ -1,9 +1,17 @@
+@use '../../partials/dir';
 @use '_scrollbar';
 
+@mixin paddingForSafari($padding) {
+       @media not all and (min-resolution:.001dpcm) {
+    @include dir.side(padding, 0, $padding);
+  }
+}
+
 .scrollbar {
   width: 100%;
   height: 100%;
   @include scrollbar.scroll;
+  @include paddingForSafari(var(--sp-extra-tight));
 
   &__h {
     @include scrollbar.scroll__h;
@@ -18,5 +26,6 @@
   }
   &--invisible {
     @include scrollbar.scroll--invisible;
+    @include paddingForSafari(0);
   }
 }
\ No newline at end of file

diff --git a/src/app/atoms/scroll/_scrollbar.scss b/src/app/atoms/scroll/_scrollbar.scss
index 11c2dc89f520e0c15a48adf093d29a87baa1fee6..78ec75ad201c34a25eb2125f70202b5fe7982988 100644 (file)
--- a/src/app/atoms/scroll/_scrollbar.scss
+++ b/src/app/atoms/scroll/_scrollbar.scss
@@ -1,5 +1,3 @@
-@use '../../partials/dir';
-
 .firefox-scrollbar {
   scrollbar-width: thin;
   scrollbar-color: var(--bg-surface-hover) transparent;
@@ -32,12 +30,6 @@
   }
 }
 
-@mixin paddingForSafari($padding) {
-       @media not all and (min-resolution:.001dpcm) {
-    @include dir.side(padding, 0, $padding);
-  }
-}
-
 @mixin scroll {
   overflow: hidden;
   // Below code stop scroll when x-scrollable content come in timeline
@@ -46,7 +38,6 @@
   @extend .webkit-scrollbar;
   @extend .webkit-scrollbar-track;
   @extend .webkit-scrollbar-thumb;
-  @include paddingForSafari(var(--sp-extra-tight));
 }
 
 @mixin scroll__h {
@@ -67,7 +58,6 @@
 @mixin scroll--invisible {
   -ms-overflow-style: none;
   scrollbar-width: none;
-  @include paddingForSafari(0);
 
   &::-webkit-scrollbar {
     display: none;