@automattic/vip-design-system 0.9.2 → 0.9.5

package/build/system/Timeline/Timeline.stories.js

@@ -1,29 +1,52 @@
+"use strict";
+
+exports.__esModule = true;
+exports.Default = exports["default"] = void 0;
+
+var _react = _interopRequireDefault(require("react"));
+
+var _ = require("..");
+
+var _jsxRuntime = require("theme-ui/jsx-runtime");
+
+function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { "default": obj }; }
+
 /**
  * External dependencies
  */
-import React from 'react';
+
 /**
  * Internal dependencies
  */
-
-import { Timeline, Grid, Box } from '..';
-export default {
+var _default = {
   title: 'Timeline',
-  component: Timeline
+  component: _.Timeline
 };
-export var Default = function Default() {
-  return /*#__PURE__*/React.createElement(React.Fragment, null, /*#__PURE__*/React.createElement(Grid, {
-    gap: 2,
-    columns: [2, '100px 300px'],
-    sx: {
-      mb: '20px'
-    }
-  }, /*#__PURE__*/React.createElement(Timeline, {
-    time: "7pm"
-  }), /*#__PURE__*/React.createElement(Box, null, "It looks like everything is recovered now. You can forget about the last event.")), /*#__PURE__*/React.createElement(Grid, {
-    gap: 2,
-    columns: [2, '100px 200px']
-  }, /*#__PURE__*/React.createElement(Timeline, {
-    time: "6pm"
-  }), /*#__PURE__*/React.createElement(Box, null, "At this particular moment in the day, something happened with your environment.")));
-};
+exports["default"] = _default;
+
+var Default = function Default() {
+  return (0, _jsxRuntime.jsxs)(_react["default"].Fragment, {
+    children: [(0, _jsxRuntime.jsxs)(_.Grid, {
+      gap: 2,
+      columns: [2, '100px 300px'],
+      sx: {
+        mb: '20px'
+      },
+      children: [(0, _jsxRuntime.jsx)(_.Timeline, {
+        time: "7pm"
+      }), (0, _jsxRuntime.jsx)(_.Box, {
+        children: "It looks like everything is recovered now. You can forget about the last event."
+      })]
+    }), (0, _jsxRuntime.jsxs)(_.Grid, {
+      gap: 2,
+      columns: [2, '100px 200px'],
+      children: [(0, _jsxRuntime.jsx)(_.Timeline, {
+        time: "6pm"
+      }), (0, _jsxRuntime.jsx)(_.Box, {
+        children: "At this particular moment in the day, something happened with your environment."
+      })]
+    })]
+  });
+};
+
+exports.Default = Default;

package/build/system/Timeline/index.js

@@ -1,71 +1,7 @@
 "use strict";
 
 exports.__esModule = true;
-exports.Timeline = void 0;
 
-var _themeUi = require("theme-ui");
+var _Timeline = require("./Timeline");
 
-var _md = require("react-icons/md");
-
-var _propTypes = _interopRequireDefault(require("prop-types"));
-
-var _jsxRuntime = require("theme-ui/jsx-runtime");
-
-var _excluded = ["time", "first", "last"];
-
-function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { "default": obj }; }
-
-function _extends() { _extends = Object.assign || function (target) { for (var i = 1; i < arguments.length; i++) { var source = arguments[i]; for (var key in source) { if (Object.prototype.hasOwnProperty.call(source, key)) { target[key] = source[key]; } } } return target; }; return _extends.apply(this, arguments); }
-
-function _objectWithoutPropertiesLoose(source, excluded) { if (source == null) return {}; var target = {}; var sourceKeys = Object.keys(source); var key, i; for (i = 0; i < sourceKeys.length; i++) { key = sourceKeys[i]; if (excluded.indexOf(key) >= 0) continue; target[key] = source[key]; } return target; }
-
-var VerticalLine = function VerticalLine() {
-  return (0, _jsxRuntime.jsx)("div", {
-    sx: {
-      borderLeft: '2px solid',
-      borderColor: 'border',
-      height: 'calc( 50% - 16px )',
-      borderRadius: '2px'
-    }
-  });
-};
-
-var Timeline = function Timeline(_ref) {
-  var time = _ref.time,
-      _ref$first = _ref.first,
-      first = _ref$first === void 0 ? false : _ref$first,
-      _ref$last = _ref.last,
-      last = _ref$last === void 0 ? false : _ref$last,
-      props = _objectWithoutPropertiesLoose(_ref, _excluded);
-
-  return (0, _jsxRuntime.jsxs)(_themeUi.Flex, _extends({}, props, {
-    children: [(0, _jsxRuntime.jsxs)(_themeUi.Flex, {
-      sx: {
-        flexDirection: 'column',
-        justifyContent: 'space-evenly',
-        alignItems: 'center'
-      },
-      children: [!first && (0, _jsxRuntime.jsx)(VerticalLine, {}), (0, _jsxRuntime.jsx)(_md.MdWatchLater, {
-        sx: {
-          color: 'border'
-        },
-        size: 18
-      }), !last && (0, _jsxRuntime.jsx)(VerticalLine, {})]
-    }), (0, _jsxRuntime.jsx)(_themeUi.Flex, {
-      sx: {
-        alignItems: 'center',
-        ml: 2
-      },
-      children: (0, _jsxRuntime.jsx)("span", {
-        children: time
-      })
-    })]
-  }));
-};
-
-exports.Timeline = Timeline;
-Timeline.propTypes = {
-  first: _propTypes["default"].bool,
-  time: _propTypes["default"].string,
-  last: _propTypes["default"].bool
-};
+exports.Timeline = _Timeline.Timeline;

package/build/system/Tooltip/Tooltip.stories.js

@@ -1,29 +1,51 @@
+"use strict";
+
+exports.__esModule = true;
+exports.Default = exports["default"] = void 0;
+
+var _react = _interopRequireDefault(require("react"));
+
+var _ = require("..");
+
+var _jsxRuntime = require("theme-ui/jsx-runtime");
+
+function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { "default": obj }; }
+
 /**
  * External dependencies
  */
-import React from 'react';
+
 /**
  * Internal dependencies
  */
-
-import { Flex, Heading, Tooltip, Text, Link } from '..';
-export default {
+var _default = {
   title: 'Tooltip',
-  component: Tooltip
+  component: _.Tooltip
 };
-export var Default = function Default() {
-  return /*#__PURE__*/React.createElement(Flex, {
+exports["default"] = _default;
+
+var Default = function Default() {
+  return (0, _jsxRuntime.jsxs)(_.Flex, {
     sx: {
       alignItems: 'center'
-    }
-  }, /*#__PURE__*/React.createElement(Heading, {
-    sx: {
-      mb: 0,
-      mr: 2
-    }
-  }, "My Section Heading"), /*#__PURE__*/React.createElement(Tooltip, null, /*#__PURE__*/React.createElement(Text, {
-    sx: {
-      fontSize: 1
-    }
-  }, "This is a tooltip that can be used to describe various pieces of functionality.", /*#__PURE__*/React.createElement("br", null), /*#__PURE__*/React.createElement(Link, null, "Find out more \u2192"))));
-};
+    },
+    children: [(0, _jsxRuntime.jsx)(_.Heading, {
+      sx: {
+        mb: 0,
+        mr: 2
+      },
+      children: "My Section Heading"
+    }), (0, _jsxRuntime.jsx)(_.Tooltip, {
+      children: (0, _jsxRuntime.jsxs)(_.Text, {
+        sx: {
+          fontSize: 1
+        },
+        children: ["This is a tooltip that can be used to describe various pieces of functionality.", (0, _jsxRuntime.jsx)("br", {}), (0, _jsxRuntime.jsx)(_.Link, {
+          children: "Find out more \u2192"
+        })]
+      })
+    })]
+  });
+};
+
+exports.Default = Default;

package/build/system/Wizard/Wizard.stories.js

@@ -1,24 +1,43 @@
+"use strict";
+
+exports.__esModule = true;
+exports.Default = exports["default"] = void 0;
+
+var _react = _interopRequireDefault(require("react"));
+
+var _ = require("..");
+
+var _jsxRuntime = require("theme-ui/jsx-runtime");
+
+function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { "default": obj }; }
+
 /**
  * External dependencies
  */
-import React from 'react';
+
 /**
  * Internal dependencies
  */
-
-import { Wizard, Flex, Box, Heading, Label, Input, Button } from '..';
-export default {
+var _default = {
   title: 'Wizard',
-  component: Wizard
+  component: _.Wizard
 };
-export var Default = function Default() {
+exports["default"] = _default;
+
+var Default = function Default() {
   var steps = [{
     title: 'Choose Domain',
     subTitle: 'You can bring a domain name you already own, or buy a new one.',
-    children: /*#__PURE__*/React.createElement(Box, null, /*#__PURE__*/React.createElement(Label, null, "Domain"), /*#__PURE__*/React.createElement(Input, {
-      autoFocus: true,
-      placeholder: "yourdomain.com"
-    }), /*#__PURE__*/React.createElement(Button, null, "Continue"))
+    children: (0, _jsxRuntime.jsxs)(_.Box, {
+      children: [(0, _jsxRuntime.jsx)(_.Label, {
+        children: "Domain"
+      }), (0, _jsxRuntime.jsx)(_.Input, {
+        autoFocus: true,
+        placeholder: "yourdomain.com"
+      }), (0, _jsxRuntime.jsx)(_.Button, {
+        children: "Continue"
+      })]
+    })
   }, {
     title: 'Configure DNS'
   }, {
@@ -26,30 +45,39 @@ export var Default = function Default() {
   }, {
     title: 'Verify Domain'
   }];
-  return /*#__PURE__*/React.createElement(React.Fragment, null, /*#__PURE__*/React.createElement(Flex, {
-    sx: {
-      alignItems: 'center'
-    }
-  }, /*#__PURE__*/React.createElement(Box, {
-    sx: {
-      flex: '1 1 auto'
-    }
-  }, /*#__PURE__*/React.createElement(Heading, {
-    variant: "h1",
-    sx: {
-      display: 'flex',
-      alignItems: 'center',
-      mb: 1
-    }
-  }, "Add Domain:", ' ', /*#__PURE__*/React.createElement("span", {
-    sx: {
-      color: 'muted',
-      ml: 2
-    }
-  }, "Production")))), /*#__PURE__*/React.createElement(Box, {
-    mt: 4
-  }, /*#__PURE__*/React.createElement(Wizard, {
-    activeStep: 0,
-    steps: steps
-  })));
-};
+  return (0, _jsxRuntime.jsxs)(_react["default"].Fragment, {
+    children: [(0, _jsxRuntime.jsx)(_.Flex, {
+      sx: {
+        alignItems: 'center'
+      },
+      children: (0, _jsxRuntime.jsx)(_.Box, {
+        sx: {
+          flex: '1 1 auto'
+        },
+        children: (0, _jsxRuntime.jsxs)(_.Heading, {
+          variant: "h1",
+          sx: {
+            display: 'flex',
+            alignItems: 'center',
+            mb: 1
+          },
+          children: ["Add Domain:", ' ', (0, _jsxRuntime.jsx)("span", {
+            sx: {
+              color: 'muted',
+              ml: 2
+            },
+            children: "Production"
+          })]
+        })
+      })
+    }), (0, _jsxRuntime.jsx)(_.Box, {
+      mt: 4,
+      children: (0, _jsxRuntime.jsx)(_.Wizard, {
+        activeStep: 0,
+        steps: steps
+      })
+    })]
+  });
+};
+
+exports.Default = Default;

package/package.json

@@ -1,12 +1,13 @@
 {
 	"name": "@automattic/vip-design-system",
-	"version": "0.9.2",
+	"version": "0.9.5",
 	"main": "build/system/index.js",
 	"scripts": {
 		"build-storybook": "build-storybook",
 		"build": "cross-env NODE_ENV=production babel src --out-dir build",
 		"format": "eslint . --fix",
 		"jest": "NODE_ENV=test jest --detectOpenHandles",
+		"jest:coverage": "npm run jest && open-cli ./coverage/index.html",
 		"jest:watch": "npm run jest --watch",
 		"lint": "eslint . -f json | eslines --quiet",
 		"storybook": "start-storybook -p 6006",
@@ -20,7 +21,8 @@
 		"babel-loader": "^8.2.2",
 		"framer-motion": "^3.9.1",
 		"react-icons": "^4.2.0",
-		"react-select": "^4.3.1"
+		"react-select": "^4.3.1",
+		"react-select-async-paginate": "^0.6.1"
 	},
 	"peerDependencies": {
 		"react": "*",
@@ -34,12 +36,21 @@
 		"**/react-dom": "17.0.1"
 	},
 	"jest": {
+		"collectCoverage": true,
+		"coverageReporters": [
+			"json",
+			"html"
+		],
 		"testURL": "http://localhost",
+		"roots": [
+			"<rootDir>/src",
+			"<rootDir>/test"
+		],
 		"setupFiles": [
-			"./test/setupTests.js"
+			"<rootDir>/test/setupTests.js"
 		],
 		"setupFilesAfterEnv": [
-			"./test/setupAfterEnv.js"
+			"<rootDir>/test/setupAfterEnv.js"
 		],
 		"transformIgnorePatterns": [
 			".*.stories.js$"
@@ -49,7 +60,6 @@
 		}
 	},
 	"devDependencies": {
-		"immer": ">=9.0.6",
 		"@axe-core/react": "4.3.2",
 		"@babel/cli": "^7.14.3",
 		"@babel/core": "7.14.0",
@@ -85,7 +95,9 @@
 		"eslint-plugin-no-async-foreach": "0.1.1",
 		"eslint-plugin-react": "7.25.3",
 		"eslint-plugin-wpcalypso": "4.1.0",
+		"immer": ">=9.0.6",
 		"jest-axe": "5.0.1",
+		"open-cli": "^7.0.1",
 		"optimize-css-assets-webpack-plugin": "^6.0.1",
 		"react-input-autosize": "^3.0.0",
 		"react-refresh": "^0.9.0",

package/src/system/Avatar/Avatar.test.js

@@ -10,22 +10,21 @@ import { axe } from 'jest-axe';
 import { Avatar } from './Avatar';
 
 describe( '<Avatar />', () => {
-	it( 'renders the Avatar without an image', () => {
-		render( <Avatar name="John Doe" /> );
+	it( 'renders the Avatar without an image', async () => {
+		const { container } = render( <Avatar name="John Doe" /> );
 
 		expect( screen.getByText( 'J' ) ).toBeInTheDocument();
-	} );
-
-	it( 'renders the Avatar with image', () => {
-		render( <Avatar name="John Doe" src="path/to/image" /> );
 
-		expect( screen.getByAltText( 'Avatar image from John Doe' ) ).toBeInTheDocument();
+		// Check for accessibility issues
+		await expect( await axe( container ) ).toHaveNoViolations();
 	} );
 
-	it( 'should not have basic accessibility issues', async () => {
+	it( 'renders the Avatar with image', async () => {
 		const { container } = render( <Avatar name="John Doe" src="path/to/image" /> );
-		const results = await axe( container );
 
-		expect( results ).toHaveNoViolations();
+		expect( screen.getByAltText( 'Avatar image from John Doe' ) ).toBeInTheDocument();
+
+		// Check for accessibility issues
+		await expect( await axe( container ) ).toHaveNoViolations();
 	} );
 } );

package/src/system/Badge/Badge.test.js

@@ -0,0 +1,30 @@
+/**
+ * External dependencies
+ */
+import { render, screen } from '@testing-library/react';
+import { axe } from 'jest-axe';
+
+/**
+ * Internal dependencies
+ */
+import { Badge } from './Badge';
+
+describe( '<Badge />', () => {
+	it( 'renders the Badge component', async () => {
+		const { container } = render( <Badge>Badge text</Badge> );
+
+		expect( screen.getByText( 'Badge text' ) ).toBeInTheDocument();
+
+		// Check for accessibility issues
+		await expect( await axe( container ) ).toHaveNoViolations();
+	} );
+
+	it( 'renders the Badge component with a different variant', async () => {
+		const { container } = render( <Badge variant="primary">Badge text</Badge> );
+
+		expect( screen.getByText( 'Badge text' ) ).toBeInTheDocument();
+
+		// Check for accessibility issues
+		await expect( await axe( container ) ).toHaveNoViolations();
+	} );
+} );

package/src/system/BlankState/BlankState.js

@@ -10,10 +10,17 @@ import PropTypes from 'prop-types';
  */
 import { Box, Heading, Text } from '..';
 
-const BlankState = ( { image, icon, title, body, cta } ) => {
+const BlankState = ( {
+	body,
+	cta,
+	icon,
+	image,
+	imageAlt = 'Image representing the blank state',
+	title,
+} ) => {
 	return (
 		<Box sx={{ textAlign: 'center', padding: 5 }}>
-			{icon ? icon : <img src={image} sx={{ mb: 3 }} alt="Icon representing the blank state" />}
+			{icon ? icon : <img src={image} sx={{ mb: 3 }} alt={imageAlt} />}
 			<Heading variant="h4">{title}</Heading>
 			<Text>{body}</Text>
 			<Box sx={{ mt: 3 }}>{cta}</Box>
@@ -22,11 +29,12 @@ const BlankState = ( { image, icon, title, body, cta } ) => {
 };
 
 BlankState.propTypes = {
-	image: PropTypes.oneOfType( [ PropTypes.object, PropTypes.string ] ),
-	icon: PropTypes.node,
-	title: PropTypes.node,
 	body: PropTypes.node,
 	cta: PropTypes.node,
+	icon: PropTypes.node,
+	image: PropTypes.oneOfType( [ PropTypes.object, PropTypes.string ] ),
+	imageAlt: PropTypes.string,
+	title: PropTypes.node,
 };
 
 export { BlankState };

package/src/system/BlankState/BlankState.test.js

@@ -0,0 +1,58 @@
+/**
+ * External dependencies
+ */
+import { render, screen } from '@testing-library/react';
+import { axe } from 'jest-axe';
+import { MdContentCopy } from 'react-icons/md';
+
+/**
+ * Internal dependencies
+ */
+import { BlankState } from './BlankState';
+import { Link } from '../Link';
+
+// eslint-disable-next-line max-len
+const image = "data:image/svg+xml,%3Csvg xmlns='http://www.w3.org/2000/svg' width='79' height='79' viewBox='0 0 79 79' fill='none'%3E%3Cpath d='M66.3001 15.9C66.3001 15.9 66.3 15.8 66.2 15.8C66.1 15.7 66 15.5 65.9001 15.4C65.9001 15.4 65.9001 15.4 65.9001 15.3L44.4001 0.2C44.3001 0.0999998 44.2 0.0999994 44.1 0.0999994C44 0.0999994 44.0001 0 43.9001 0H43.3C43.2 0 43.1 0.0999994 43.1 0.0999994C43 0.0999994 42.9 0.2 42.8 0.2L28.2001 10.3C28.1 10.4 28.0001 10.5 27.9001 10.6C27.9001 10.6 27.9001 10.6 27.9001 10.7C27.8001 10.8 27.7001 11 27.7001 11.2C27.7001 11.2 27.7001 11.2 27.7001 11.3V31.1L13.9 40.8L13.7001 41C13.6001 41.1 13.6 41.1 13.6 41.2L13.5 41.3C13.5 41.4 13.4 41.5 13.4 41.5V41.6C13.4 41.7 13.3 41.8 13.3 42V62.2C13.3 62.4 13.3 62.6 13.4 62.7V62.8C13.5 62.9 13.6001 63.1 13.7001 63.2C13.7001 63.2 13.7 63.2 13.8 63.3L13.9 63.4L35.3 78.6H35.4001C35.4001 78.6 35.5 78.6 35.5 78.7H35.6C35.8 78.8 36 78.8 36.2001 78.8C36.3001 78.8 36.5 78.8 36.6 78.7H36.7001C36.8001 78.7 36.8001 78.7 36.9001 78.6C36.9001 78.6 37 78.6 37 78.5H37.1L66 58.3C66.1 58.2 66.2001 58.1 66.3001 58V15.9ZM34.6 74.5L16.1 61.3V44.7L34.6 57.8V74.5ZM36 55.3L17.2001 41.9L29 33.5L47.9001 46.9L36 55.3ZM49 44.1L30.5 31V14.4L49 27.5V44.1ZM50.5 24.9L31.6 11.5L43.5 3.2L62.4001 16.6L50.5 24.9Z' fill='%23BD9D70'/%3E%3C/svg%3E";
+
+const defaultProps = {
+	body: 'Sorry, there\'s nothing here yet.',
+	cta: <Link as="a">Explore add-ons →</Link>,
+	image,
+	imageAlt: 'This is the image alt',
+	title: 'Power up your application',
+};
+
+describe( '<BlankState />', () => {
+	it( 'renders the BlankState component', async () => {
+		const { container } = render( <BlankState { ...defaultProps } /> );
+
+		expect( screen.getByText( defaultProps.body ) ).toBeInTheDocument();
+		expect( screen.getByText( defaultProps.title ) ).toBeInTheDocument();
+		expect( screen.getByText( 'Explore add-ons →' ) ).toBeInTheDocument();
+		expect( screen.getByAltText( defaultProps.imageAlt ) ).toBeInTheDocument();
+
+		// Check for accessibility issues
+		await expect( await axe( container ) ).toHaveNoViolations();
+	} );
+
+	it( 'renders the BlankState component with default alt text for the given image', async () => {
+		const props = { ...defaultProps, imageAlt: undefined };
+		const { container } = render( <BlankState { ...props } /> );
+
+		expect( screen.getByAltText( 'Image representing the blank state' ) ).toBeInTheDocument();
+
+		// Check for accessibility issues
+		await expect( await axe( container ) ).toHaveNoViolations();
+	} );
+
+	it( 'renders the BlankState component with an icon', async () => {
+		const icon = <MdContentCopy title="this is an icon" />;
+		const props = { ...defaultProps, icon };
+		const { container } = render( <BlankState { ...props } /> );
+
+		expect( screen.getByTitle( 'this is an icon' ) ).toBeInTheDocument();
+
+		// Check for accessibility issues
+		await expect( await axe( container ) ).toHaveNoViolations();
+	} );
+} );

package/src/system/Button/Button.test.js

@@ -0,0 +1,21 @@
+/**
+ * External dependencies
+ */
+import { render, screen } from '@testing-library/react';
+import { axe } from 'jest-axe';
+
+/**
+ * Internal dependencies
+ */
+import { Button } from './Button';
+
+describe( '<Button />', () => {
+	it( 'renders the Button component', async () => {
+		const { container } = render( <Button>Button text</Button> );
+
+		expect( screen.getByText( 'Button text' ) ).toBeInTheDocument();
+
+		// Check for accessibility issues
+		await expect( await axe( container ) ).toHaveNoViolations();
+	} );
+} );

package/src/system/Card/Card.test.js

@@ -0,0 +1,33 @@
+/**
+ * External dependencies
+ */
+import { render, screen, fireEvent, waitFor } from '@testing-library/react';
+import { axe } from 'jest-axe';
+import { MdContentCopy } from 'react-icons/md';
+
+/**
+ * Internal dependencies
+ */
+import { Card } from './Card';
+
+const defaultProps = {};
+
+describe( '<Card />', () => {
+	it( 'renders the Card component', async () => {
+		const { container } = render( <Card { ...defaultProps }>This is a Card</Card> );
+
+		expect( screen.getByText( 'This is a Card' ) ).toBeInTheDocument();
+
+		// Check for accessibility issues
+		await expect( await axe( container ) ).toHaveNoViolations();
+	} );
+
+	it( 'renders the Card component with a different variant', async () => {
+		const { container } = render( <Card variant="primary">Card text</Card> );
+
+		expect( screen.getByText( 'Card text' ) ).toBeInTheDocument();
+
+		// Check for accessibility issues
+		await expect( await axe( container ) ).toHaveNoViolations();
+	} );
+} );

package/src/system/Code/Code.js

@@ -47,18 +47,21 @@ const Code = ( { prompt = false, showCopy = false, onCopy = null, ...props } ) =
 		>
 			{ codeDom }
 			{
-				<span
+				<button
+					aria-label="Copy"
 					sx={ {
-						position: 'absolute',
-						top: 0,
 						bg: 'grey.10',
-						right: 0,
-						paddingRight: 2,
-						paddingLeft: 2,
-						paddingTop: 1,
-						paddingBottom: 1,
 						borderTopRightRadius: 1,
+						borderWidth: 0,
+						color: 'muted',
 						opacity: 0.8,
+						paddingBottom: 1,
+						paddingLeft: 2,
+						paddingRight: 2,
+						paddingTop: 1,
+						position: 'absolute',
+						right: 0,
+						top: '-1px',
 						'&:hover': {
 							opacity: 1,
 							cursor: 'pointer',
@@ -75,7 +78,7 @@ const Code = ( { prompt = false, showCopy = false, onCopy = null, ...props } ) =
 					} }
 				>
 					{ copied ? 'Copied!' : <MdContentCopy /> }
-				</span>
+				</button>
 			}
 		</div>
 	);

package/src/system/Code/Code.stories.js

@@ -16,7 +16,9 @@ export default {
 export const Default = () => (
 	<React.Fragment>
 		<Code>Code</Code>
+		<br />
 		<Code showCopy={true}>Code with Icon</Code>
+		<br />
 		<Code showCopy={true} onCopy={
 			// eslint-disable-next-line no-console
 			() => console.info( 'Hello world' )