update snapshots, remove viewport

addons/storyshots/src/test-body-image-snapshot.js

@@ -7,17 +7,13 @@ expect.extend({ toMatchImageSnapshot });
 // We consider taking the full page is a reasonnable default.
 const defaultScreenshotOptions = () => ({ fullPage: true });
 
-// Enforce default viewport to have a minimal width & height for screenshots.
-// Helps avoiding image size inconsistency thus not generating image diff.
-const defaultBeforeScreenshot = page => {
-  page.setViewport({ width: 1024, height: 500 });
-};
+const noop = () => {};
 
 export const imageSnapshot = ({
   storybookUrl = 'http://localhost:6006',
-  getMatchOptions = () => {},
+  getMatchOptions = noop,
   getScreenshotOptions = defaultScreenshotOptions,
-  beforeScreenshot = defaultBeforeScreenshot,
+  beforeScreenshot = noop,
 }) => {
   let browser; // holds ref to browser. (ie. Chrome)
   let page; // Hold ref to the page to screenshot.

examples/official-storybook/image-snapshots/storyshots-image.runner.js

@@ -24,11 +24,6 @@ if (!fs.existsSync(pathToStorybookStatic)) {
     test: imageSnapshot({
       storybookUrl: `file://${pathToStorybookStatic}`,
       beforeScreenshot: (page, { context }) => {
-        // Default viewport set. The height is not restrictive since we tell Chrome to take the 'fullPage' screenshot.
-        // Also, setViewport does not accept setting a width and not an height.
-        // Finally, setting an explicit height is useful to generate diffs even if there is no content.
-        page.setViewport({ width: 1024, height: 500 });
-
         // Screenshots for this story have to be tweaked.
         if (context.kind === 'ui/Layout') {
           // Some weird stuff are done inside Layout component, so we have to wait some to get a viable screenshot.