fix(gatsby-image): do not render the source tag if no srcSet is provi…

…ded (#26766)
gatsbyjs · Sep 3, 2020 · 268d982 · 268d982
1 parent 123b19f
commit 268d982
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/packages/gatsby-image/src/__tests__/index.js b/packages/gatsby-image/src/__tests__/index.js
@@ -369,4 +369,16 @@ describe(`<Image />`, () => {
     const placeholderImageTag = setup().querySelector(`picture img`)
     expect(placeholderImageTag.getAttribute(`aria-hidden`)).toBe(null)
   })
+
+  it(`should not have a "source" tag if no srcSet is provided`, () => {
+    jest.spyOn(global.console, `warn`)
+
+    const props = {
+      fixed: { ...fixedShapeMock, srcSet: null, srcSetWebp: null },
+    }
+    const sourceTag = setup(false, props).querySelector(`source`)
+    expect(sourceTag).toEqual(null)
+
+    expect(console.warn).toBeCalled()
+  })
 })
diff --git a/packages/gatsby-image/src/index.js b/packages/gatsby-image/src/index.js
@@ -176,7 +176,7 @@ function generateImageSources(imageVariants) {
           sizes={sizes}
         />
       )}
-      <source media={media} srcSet={srcSet} sizes={sizes} />
+      {srcSet && <source media={media} srcSet={srcSet} sizes={sizes} />}
     </React.Fragment>
   ))
 }