fix: added missing logical props

Kajabi · Sep 30, 2024 · b7c17a3 · b7c17a3
1 parent 976887d
commit b7c17a3
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 11 deletions.
diff --git a/docs/lib/sage-frontend/stylesheets/docs/_icon.scss b/docs/lib/sage-frontend/stylesheets/docs/_icon.scss
@@ -9,7 +9,7 @@
   flex-direction: column;
   flex-wrap: wrap;
   align-items: center;
-  margin-bottom: sage-spacing();
+  margin-block-end: sage-spacing();
   padding: sage-spacing(lg) sage-spacing(sm);
   text-align: center;
   background: sage-color(grey, 200);
@@ -18,15 +18,15 @@
 }
 
 .docs-icon-block__label {
-  margin-top: sage-spacing(xs);
-  margin-bottom: 0;
+  margin-block-start: sage-spacing(xs);
+  margin-block-end: 0;
   font-size: sage-font-size(body-xs);
   font-weight: sage-font-weight(semibold);
 }
 
 
 .docs-icon-inline {
-  margin-bottom: sage-spacing(sm);
+  margin-block-end: sage-spacing(sm);
   padding: sage-spacing(sm);
   text-align: right;
   background: sage-color(grey, 200);

diff --git a/packages/sage-assets/lib/stylesheets/components/_choice.scss b/packages/sage-assets/lib/stylesheets/components/_choice.scss
@@ -135,7 +135,7 @@ $-choice-radio-color-checked-inner: map-get($sage-radio-colors, checked-inner);
   }
 
   pds-icon {
-    margin-top: var(--icon-top-offset);
+    margin-block-start: var(--icon-top-offset);
   }
 
   &.sage-choice--vertical-align-icon-start::#{$-choice-el-icon} {

diff --git a/packages/sage-assets/lib/stylesheets/components/_empty_state.scss b/packages/sage-assets/lib/stylesheets/components/_empty_state.scss
@@ -11,8 +11,8 @@ $-empty-state-graphic-page-height: rem(336px);
 $-empty-state-page-max-width: rem(1064px);
 
 .sage-empty-state {
-  margin-left: auto;
-  margin-right: auto;
+  margin-inline-start: auto;
+  margin-inline-end: auto;
 
   &:not(.sage-empty-state--page) {
     @include sage-grid-panel();
@@ -65,7 +65,7 @@ $-empty-state-page-max-width: rem(1064px);
 .sage-empty-state__actions,
 .sage-empty-state__custom-content,
 .sage-empty-state__video {
-  margin-top: sage-spacing();
+  margin-block-start: sage-spacing();
 }
 
 .sage-empty-state__content {
@@ -119,7 +119,7 @@ $-empty-state-page-max-width: rem(1064px);
   }
 
   &:not(:only-child) {
-    margin-bottom: sage-spacing(sm);
+    margin-block-end: sage-spacing(sm);
   }
 }
 

diff --git a/packages/sage-assets/lib/stylesheets/components/_icon.scss b/packages/sage-assets/lib/stylesheets/components/_icon.scss
@@ -75,7 +75,7 @@ $-icon-beside-type: (
 
   .sage-tab--icon-#{$icon-name}::before {
     @include sage-icon-base($icon-name, md, pine);
-    margin-right: sage-spacing(xs);
+    margin-inline-end: sage-spacing(xs);
   }
 
   .sage-choice--icon-#{$icon-name}::before {
@@ -115,7 +115,7 @@ $-icon-beside-type: (
     white-space: nowrap;
 
     &::before {
-      margin-right: sage-spacing(xs);
+      margin-inline-end: sage-spacing(xs);
 
       @include sage-icon-base($icon-name, md, pine);
     }