now with functioning tree-shaking

gatsbyjs · Jun 30, 2020 · ed25dd1 · ed25dd1
1 parent 74b7c3b
commit ed25dd1
Showing 1 changed file with 11 additions and 9 deletions.
diff --git a/packages/gatsby-image/src/index.js b/packages/gatsby-image/src/index.js
@@ -330,12 +330,22 @@ Img.propTypes = {
   onLoad: PropTypes.func,
 }
 
+// Intentionally not an instance method because webpack is not able to
+// three-shake it away in production
+function validateImageProps({ fluid, fixed }) {
+  if (!fluid && !fixed) {
+    console.warn(
+      `gatsby-image expects a 'fixed' or a 'fluid' prop; neither was present.`
+    )
+  }
+}
+
 class Image extends React.Component {
   constructor(props) {
     super(props)
 
     if (process.env.NODE_ENV !== `production`) {
-      this.validateProps(props)
+      validateImageProps(props)
     }
 
     // If this image has already been loaded before then we can assume it's
@@ -368,14 +378,6 @@ class Image extends React.Component {
     this.handleRef = this.handleRef.bind(this)
   }
 
-  validateProps({ fluid, fixed }) {
-    if (!fluid && !fixed) {
-      console.warn(
-        `gatsby-image expects a 'fixed' or a 'fluid' prop; neither was present.`
-      )
-    }
-  }
-
   componentDidMount() {
     if (this.state.isVisible && typeof this.props.onStartLoad === `function`) {
       this.props.onStartLoad({ wasCached: inImageCache(this.props) })