fix: only fire warnings during development

kripod · Apr 4, 2020 · 239edda · 239edda
1 parent adca16f
commit 239edda
Showing 1 changed file with 8 additions and 6 deletions.
diff --git a/packages/glaze/src/StyleInjector.tsx b/packages/glaze/src/StyleInjector.tsx
@@ -6,7 +6,7 @@ Based on:
 
 import React from 'react';
 
-import { canUseDOM } from './env';
+import { canUseDOM, isDev } from './env';
 import { warnOnce } from './logger';
 
 const MAX_RULE_COUNT = 0xffff;
@@ -107,11 +107,13 @@ export class NullStyleInjector implements StyleInjector {
 
   // eslint-disable-next-line class-methods-use-this
   addRule(): number {
-    // TODO: Add instructions for resolving the situation
-    if (canUseDOM) {
-      warnOnce('Client-side rendering of dynamic styles is not set up');
-    } else {
-      warnOnce('Server-side rendering of dynamic styles is not set up');
+    if (isDev) {
+      // TODO: Add instructions for resolving the situation
+      if (canUseDOM) {
+        warnOnce('Client-side rendering of dynamic styles is not set up');
+      } else {
+        warnOnce('Server-side rendering of dynamic styles is not set up');
+      }
     }
 
     return 0;