fix(styles): add error handling for rem & em sass functions (#4931)

carbon-design-system · Jan 6, 2020 · 51558b6 · 51558b6
1 parent 057fade
commit 51558b6
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 0 deletions.
diff --git a/packages/components/src/globals/scss/_typography.scss b/packages/components/src/globals/scss/_typography.scss
@@ -22,6 +22,11 @@ $base-font-size: 16px !default;
 /// @group global-typography
 /// @deprecated (For v10) Use `carbon--rem()`
 @function rem($px) {
+  @if unit($px) != 'px' {
+    // TODO: update to @error in v11
+    @warn "Expected argument $px to be of type `px`, instead received: `#{unit($px)}`";
+  }
+
   @return ($px / $base-font-size) * 1rem;
 }
 
@@ -33,6 +38,11 @@ $base-font-size: 16px !default;
 /// @group global-typography
 /// @deprecated (For v10) Use `carbon--em()`
 @function em($px) {
+  @if unit($px) != 'px' {
+    // TODO: update to @error in v11
+    @warn "Expected argument $px to be of type `px`, instead received: `#{unit($px)}`";
+  }
+
   @return ($px / $base-font-size) * 1em;
 }
 

diff --git a/packages/layout/scss/_convert.scss b/packages/layout/scss/_convert.scss
@@ -17,6 +17,11 @@ $carbon--base-font-size: 16px !default;
 /// @access public
 /// @group @carbon/layout
 @function carbon--rem($px) {
+  @if unit($px) != 'px' {
+    // TODO: update to @error in v11
+    @warn "Expected argument $px to be of type `px`, instead received: `#{unit($px)}`";
+  }
+
   @return ($px / $carbon--base-font-size) * 1rem;
 }
 
@@ -26,5 +31,10 @@ $carbon--base-font-size: 16px !default;
 /// @access public
 /// @group @carbon/layout
 @function carbon--em($px) {
+  @if unit($px) != 'px' {
+    // TODO: update to @error in v11
+    @warn "Expected argument $px to be of type `px`, instead received: `#{unit($px)}`";
+  }
+
   @return ($px / $carbon--base-font-size) * 1em;
 }