Don't process constant() and env(); fixes MoOx#34

dlee · Sep 17, 2018 · 63e5da1 · 63e5da1 · sbaechler · Sep 25, 2018
1 parent 00eb6f3
commit 63e5da1
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 0 deletions.
diff --git a/src/__tests__/index.js b/src/__tests__/index.js
@@ -338,3 +338,17 @@ test(
  'calc( (1em - calc( 10px + 1em)) / 2)',
  '-5px'
 )
+
+test(
+ 'should skip constant()',
+ testFixture,
+ 'calc(constant(safe-area-inset-left))',
+ 'calc(constant(safe-area-inset-left))'
+)
+
+test(
+ 'should skip env()',
+ testFixture,
+ 'calc(env(safe-area-inset-left))',
+ 'calc(env(safe-area-inset-left))'
+)
diff --git a/src/index.js b/src/index.js
@@ -14,6 +14,10 @@ export default (value, precision = 5) => {
 
  // stringify calc expression and produce an AST
  const contents = valueParser.stringify(node.nodes)
+
+ // skip constant() and env()
+ if (contents.indexOf('constant') >= 0 || contents.indexOf('env') >= 0) return;
+
  const ast = parser.parse(contents)
 
  // reduce AST to its simplest form, that is, either to a single value