@dxc-technology/halstack-react 0.0.0-d3554d7 → 0.0.0-d3624d0

package/button/Button.test.js

@@ -1,35 +1,38 @@
 "use strict";
 
 var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault");
-
 var _react = _interopRequireDefault(require("react"));
-
 var _react2 = require("@testing-library/react");
-
-var _Button = _interopRequireDefault(require("./Button"));
-
+var _Button = _interopRequireDefault(require("./Button.tsx"));
 describe("Button component tests", function () {
   test("Button renders with correct text", function () {
     var _render = (0, _react2.render)( /*#__PURE__*/_react["default"].createElement(_Button["default"], {
-      label: "Button"
-    })),
-        getByText = _render.getByText;
-
+        label: "Button"
+      })),
+      getByText = _render.getByText;
     expect(getByText("Button")).toBeTruthy();
   });
   test("Calls correct function on click", function () {
     var onClick = jest.fn();
-
     var _render2 = (0, _react2.render)( /*#__PURE__*/_react["default"].createElement(_Button["default"], {
-      label: "Button",
-      onClick: onClick
-    })),
-        getByText = _render2.getByText;
-
+        label: "Button",
+        onClick: onClick
+      })),
+      getByText = _render2.getByText;
     var button = getByText("Button");
-
     _react2.fireEvent.click(button);
-
     expect(onClick).toHaveBeenCalled();
   });
+  test("Renders with correct accessibility attributes", function () {
+    var _render3 = (0, _react2.render)( /*#__PURE__*/_react["default"].createElement(_Button["default"], {
+        label: "Home",
+        title: "Go home",
+        tabIndex: 1
+      })),
+      getByRole = _render3.getByRole;
+    var button = getByRole("button");
+    expect(button.getAttribute("aria-label")).toBe("Go home");
+    expect(button.getAttribute("title")).toBe("Go home");
+    expect(button.getAttribute("tabindex")).toBe("1");
+  });
 });

package/button/types.d.ts

@@ -1,19 +1,19 @@
 /// <reference types="react" />
-declare type Space = "xxsmall" | "xsmall" | "small" | "medium" | "large" | "xlarge" | "xxlarge";
-declare type Margin = {
+type Space = "xxsmall" | "xsmall" | "small" | "medium" | "large" | "xlarge" | "xxlarge";
+type Margin = {
     top?: Space;
     bottom?: Space;
     left?: Space;
     right?: Space;
 };
-declare type SVG = React.SVGProps<SVGSVGElement>;
-declare type Props = {
+type SVG = React.ReactNode & React.SVGProps<SVGSVGElement>;
+type Props = {
     /**
-     * Text to be placed next to the button.
+     * Text to be placed in the button.
      */
     label?: string;
     /**
-     * Uses one of the available button modes.
+     * The available button modes.
      */
     mode?: "primary" | "secondary" | "text";
     /**
@@ -25,11 +25,15 @@ declare type Props = {
      */
     iconPosition?: "before" | "after";
     /**
-     * This prop corresponds to the 'type' prop of the button in html.
+     * Value for the HTML properties title and aria-label.
+     */
+    title?: string;
+    /**
+     * 'type' html prop of the button.
      */
     type?: "button" | "reset" | "submit";
     /**
-     * Element or path used as the icon that will be placed next to the button label.
+     * Material Symbol name or SVG element as the icon that will be placed next to the label.
      */
     icon?: string | SVG;
     /**

package/card/Card.accessibility.test.js

@@ -0,0 +1,36 @@
+"use strict";
+
+var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault");
+var _regenerator = _interopRequireDefault(require("@babel/runtime/regenerator"));
+var _asyncToGenerator2 = _interopRequireDefault(require("@babel/runtime/helpers/asyncToGenerator"));
+var _react = _interopRequireDefault(require("react"));
+var _react2 = require("@testing-library/react");
+var _axeHelper = require("../../test/accessibility/axe-helper.js");
+var _Card = _interopRequireDefault(require("./Card.tsx"));
+describe("Card component accessibility tests", function () {
+  it("Should not have basic accessibility issues", /*#__PURE__*/(0, _asyncToGenerator2["default"])( /*#__PURE__*/_regenerator["default"].mark(function _callee() {
+    var _render, container, results;
+    return _regenerator["default"].wrap(function _callee$(_context) {
+      while (1) switch (_context.prev = _context.next) {
+        case 0:
+          _render = (0, _react2.render)( /*#__PURE__*/_react["default"].createElement(_Card["default"], {
+            linkHref: "https://www.dxc.com",
+            imageSrc: "https://picsum.photos/id/1022/200/300",
+            imagePosition: "after",
+            imagePadding: "xsmall",
+            margin: "medium",
+            imageBgColor: "yellow",
+            imageCover: true
+          }, "test-card")), container = _render.container;
+          _context.next = 3;
+          return (0, _axeHelper.axe)(container);
+        case 3:
+          results = _context.sent;
+          expect(results).toHaveNoViolations();
+        case 5:
+        case "end":
+          return _context.stop();
+      }
+    }, _callee);
+  })));
+});

package/card/Card.d.ts

@@ -1,4 +1,4 @@
 /// <reference types="react" />
 import CardPropsType from "./types";
-declare const DxcCard: ({ imageSrc, imageBgColor, imagePadding, imagePosition, linkHref, onClick, imageCover, margin, contentPadding, tabIndex, outlined, children, }: CardPropsType) => JSX.Element;
+declare const DxcCard: ({ imageSrc, imageBgColor, imagePadding, imagePosition, linkHref, onClick, imageCover, margin, tabIndex, outlined, children, }: CardPropsType) => JSX.Element;
 export default DxcCard;

package/card/Card.js

@@ -1,79 +1,46 @@
 "use strict";
 
 var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault");
-
 var _typeof3 = require("@babel/runtime/helpers/typeof");
-
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports["default"] = void 0;
-
 var _typeof2 = _interopRequireDefault(require("@babel/runtime/helpers/typeof"));
-
 var _taggedTemplateLiteral2 = _interopRequireDefault(require("@babel/runtime/helpers/taggedTemplateLiteral"));
-
 var _slicedToArray2 = _interopRequireDefault(require("@babel/runtime/helpers/slicedToArray"));
-
 var _react = _interopRequireWildcard(require("react"));
-
 var _styledComponents = _interopRequireWildcard(require("styled-components"));
-
-var _variables = require("../common/variables.js");
-
+var _variables = require("../common/variables");
 var _useTheme = _interopRequireDefault(require("../useTheme"));
-
 var _Box = _interopRequireDefault(require("../box/Box"));
-
-var _templateObject, _templateObject2, _templateObject3, _templateObject4, _templateObject5, _templateObject6;
-
-function _getRequireWildcardCache(nodeInterop) { if (typeof WeakMap !== "function") return null; var cacheBabelInterop = new WeakMap(); var cacheNodeInterop = new WeakMap(); return (_getRequireWildcardCache = function _getRequireWildcardCache(nodeInterop) { return nodeInterop ? cacheNodeInterop : cacheBabelInterop; })(nodeInterop); }
-
-function _interopRequireWildcard(obj, nodeInterop) { if (!nodeInterop && obj && obj.__esModule) { return obj; } if (obj === null || _typeof3(obj) !== "object" && typeof obj !== "function") { return { "default": obj }; } var cache = _getRequireWildcardCache(nodeInterop); if (cache && cache.has(obj)) { return cache.get(obj); } var newObj = {}; var hasPropertyDescriptor = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var key in obj) { if (key !== "default" && Object.prototype.hasOwnProperty.call(obj, key)) { var desc = hasPropertyDescriptor ? Object.getOwnPropertyDescriptor(obj, key) : null; if (desc && (desc.get || desc.set)) { Object.defineProperty(newObj, key, desc); } else { newObj[key] = obj[key]; } } } newObj["default"] = obj; if (cache) { cache.set(obj, newObj); } return newObj; }
-
+var _templateObject, _templateObject2, _templateObject3, _templateObject4, _templateObject5;
+function _getRequireWildcardCache(e) { if ("function" != typeof WeakMap) return null; var r = new WeakMap(), t = new WeakMap(); return (_getRequireWildcardCache = function _getRequireWildcardCache(e) { return e ? t : r; })(e); }
+function _interopRequireWildcard(e, r) { if (!r && e && e.__esModule) return e; if (null === e || "object" != _typeof3(e) && "function" != typeof e) return { "default": e }; var t = _getRequireWildcardCache(r); if (t && t.has(e)) return t.get(e); var n = { __proto__: null }, a = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var u in e) if ("default" !== u && {}.hasOwnProperty.call(e, u)) { var i = a ? Object.getOwnPropertyDescriptor(e, u) : null; i && (i.get || i.set) ? Object.defineProperty(n, u, i) : n[u] = e[u]; } return n["default"] = e, t && t.set(e, n), n; }
 var DxcCard = function DxcCard(_ref) {
   var imageSrc = _ref.imageSrc,
-      _ref$imageBgColor = _ref.imageBgColor,
-      imageBgColor = _ref$imageBgColor === void 0 ? "black" : _ref$imageBgColor,
-      imagePadding = _ref.imagePadding,
-      _ref$imagePosition = _ref.imagePosition,
-      imagePosition = _ref$imagePosition === void 0 ? "before" : _ref$imagePosition,
-      linkHref = _ref.linkHref,
-      onClick = _ref.onClick,
-      _ref$imageCover = _ref.imageCover,
-      imageCover = _ref$imageCover === void 0 ? false : _ref$imageCover,
-      margin = _ref.margin,
-      contentPadding = _ref.contentPadding,
-      _ref$tabIndex = _ref.tabIndex,
-      tabIndex = _ref$tabIndex === void 0 ? 0 : _ref$tabIndex,
-      _ref$outlined = _ref.outlined,
-      outlined = _ref$outlined === void 0 ? true : _ref$outlined,
-      children = _ref.children;
+    _ref$imageBgColor = _ref.imageBgColor,
+    imageBgColor = _ref$imageBgColor === void 0 ? "black" : _ref$imageBgColor,
+    imagePadding = _ref.imagePadding,
+    _ref$imagePosition = _ref.imagePosition,
+    imagePosition = _ref$imagePosition === void 0 ? "before" : _ref$imagePosition,
+    linkHref = _ref.linkHref,
+    onClick = _ref.onClick,
+    _ref$imageCover = _ref.imageCover,
+    imageCover = _ref$imageCover === void 0 ? false : _ref$imageCover,
+    margin = _ref.margin,
+    _ref$tabIndex = _ref.tabIndex,
+    tabIndex = _ref$tabIndex === void 0 ? 0 : _ref$tabIndex,
+    _ref$outlined = _ref.outlined,
+    outlined = _ref$outlined === void 0 ? true : _ref$outlined,
+    children = _ref.children;
   var colorsTheme = (0, _useTheme["default"])();
-
   var _useState = (0, _react.useState)(false),
-      _useState2 = (0, _slicedToArray2["default"])(_useState, 2),
-      isHovered = _useState2[0],
-      changeIsHovered = _useState2[1];
-
-  var tagContent = /*#__PURE__*/_react["default"].createElement(_Box["default"], {
-    shadowDepth: !outlined ? 0 : isHovered && (onClick || linkHref) ? 2 : 1
-  }, /*#__PURE__*/_react["default"].createElement(_styledComponents.ThemeProvider, {
-    theme: colorsTheme.card
-  }, /*#__PURE__*/_react["default"].createElement(CardContainer, {
-    hasAction: onClick || linkHref,
-    imagePosition: imagePosition
-  }, imageSrc && /*#__PURE__*/_react["default"].createElement(ImageContainer, {
-    imageBgColor: imageBgColor
-  }, /*#__PURE__*/_react["default"].createElement(TagImage, {
-    imagePadding: imagePadding,
-    cover: imageCover,
-    src: imageSrc
-  })), /*#__PURE__*/_react["default"].createElement(CardContent, {
-    contentPadding: contentPadding
-  }, children))));
-
-  return /*#__PURE__*/_react["default"].createElement(StyledDxcCard, {
+    _useState2 = (0, _slicedToArray2["default"])(_useState, 2),
+    isHovered = _useState2[0],
+    changeIsHovered = _useState2[1];
+  return /*#__PURE__*/_react["default"].createElement(Card, {
+    hasAction: onClick || linkHref ? true : false,
     margin: margin,
     onMouseEnter: function onMouseEnter() {
       return changeIsHovered(true);
@@ -82,15 +49,26 @@ var DxcCard = function DxcCard(_ref) {
       return changeIsHovered(false);
     },
     onClick: onClick,
-    hasAction: onClick,
-    tabIndex: typeof onClick === "function" && !linkHref ? tabIndex : -1
-  }, linkHref && /*#__PURE__*/_react["default"].createElement(StyledLink, {
-    tabIndex: tabIndex,
+    tabIndex: onClick || linkHref ? tabIndex : -1,
+    as: linkHref && "a",
     href: linkHref
-  }, tagContent) || tagContent);
+  }, /*#__PURE__*/_react["default"].createElement(_Box["default"], {
+    shadowDepth: !outlined ? 0 : isHovered && (onClick || linkHref) ? 2 : 1
+  }, /*#__PURE__*/_react["default"].createElement(_styledComponents.ThemeProvider, {
+    theme: colorsTheme.card
+  }, /*#__PURE__*/_react["default"].createElement(CardContainer, {
+    hasAction: onClick || linkHref ? true : false,
+    imagePosition: imageSrc ? imagePosition : "none"
+  }, imageSrc && /*#__PURE__*/_react["default"].createElement(ImageContainer, {
+    imageBgColor: imageBgColor
+  }, /*#__PURE__*/_react["default"].createElement(TagImage, {
+    imagePadding: imagePadding,
+    imageCover: imageCover,
+    src: imageSrc,
+    alt: "Card image"
+  })), /*#__PURE__*/_react["default"].createElement(CardContent, null, children)))));
 };
-
-var StyledDxcCard = _styledComponents["default"].div(_templateObject || (_templateObject = (0, _taggedTemplateLiteral2["default"])(["\n  display: inline-flex;\n  cursor: ", ";\n  outline: ", ";\n  margin: ", ";\n  margin-top: ", ";\n  margin-right: ", ";\n  margin-bottom: ", ";\n  margin-left: ", ";\n\n  :focus {\n    outline: #0095ff auto 1px;\n  }\n"])), function (_ref2) {
+var Card = _styledComponents["default"].div(_templateObject || (_templateObject = (0, _taggedTemplateLiteral2["default"])(["\n  display: inline-flex;\n  cursor: ", ";\n  outline: ", ";\n  margin: ", ";\n  margin-top: ", ";\n  margin-right: ", ";\n  margin-bottom: ", ";\n  margin-left: ", ";\n  text-decoration: none;\n  ", "\n"])), function (_ref2) {
   var hasAction = _ref2.hasAction;
   return hasAction && "pointer" || "unset";
 }, function (_ref3) {
@@ -101,21 +79,22 @@ var StyledDxcCard = _styledComponents["default"].div(_templateObject || (_templa
   return margin && (0, _typeof2["default"])(margin) !== "object" ? _variables.spaces[margin] : "0px";
 }, function (_ref5) {
   var margin = _ref5.margin;
-  return margin && margin.top ? _variables.spaces[margin.top] : "";
+  return margin && (0, _typeof2["default"])(margin) === "object" && margin.top ? _variables.spaces[margin.top] : "";
 }, function (_ref6) {
   var margin = _ref6.margin;
-  return margin && margin.right ? _variables.spaces[margin.right] : "";
+  return margin && (0, _typeof2["default"])(margin) === "object" && margin.right ? _variables.spaces[margin.right] : "";
 }, function (_ref7) {
   var margin = _ref7.margin;
-  return margin && margin.bottom ? _variables.spaces[margin.bottom] : "";
+  return margin && (0, _typeof2["default"])(margin) === "object" && margin.bottom ? _variables.spaces[margin.bottom] : "";
 }, function (_ref8) {
   var margin = _ref8.margin;
-  return margin && margin.left ? _variables.spaces[margin.left] : "";
+  return margin && (0, _typeof2["default"])(margin) === "object" && margin.left ? _variables.spaces[margin.left] : "";
+}, function (_ref9) {
+  var hasAction = _ref9.hasAction;
+  return hasAction && ":focus {\n      outline: #0095ff auto 1px;\n    }";
 });
-
-var CardContainer = _styledComponents["default"].div(_templateObject2 || (_templateObject2 = (0, _taggedTemplateLiteral2["default"])(["\n  display: inline-flex;\n  flex-direction: ", ";\n  height: ", ";\n  width: ", ";\n  &:hover {\n    border-color: ", ";\n  }\n"])), function (_ref9) {
-  var imagePosition = _ref9.imagePosition;
-  return imagePosition === "before" && "row" || "row-reverse";
+var CardContainer = _styledComponents["default"].div(_templateObject2 || (_templateObject2 = (0, _taggedTemplateLiteral2["default"])(["\n  display: inline-flex;\n  flex-direction: ", ";\n  height: ", ";\n  width: ", ";\n  &:hover {\n    border-color: ", ";\n  }\n"])), function (props) {
+  return props.imagePosition === "after" ? "row-reverse" : "row";
 }, function (props) {
   return props.theme.height;
 }, function (props) {
@@ -124,41 +103,19 @@ var CardContainer = _styledComponents["default"].div(_templateObject2 || (_templ
   var hasAction = _ref10.hasAction;
   return hasAction ? "" : "unset";
 });
-
-var StyledLink = _styledComponents["default"].a(_templateObject3 || (_templateObject3 = (0, _taggedTemplateLiteral2["default"])(["\n  cursor: pointer;\n  text-decoration: none;\n\n  :focus {\n    outline-color: #0095ff;\n  }\n"])));
-
-var TagImage = _styledComponents["default"].img(_templateObject4 || (_templateObject4 = (0, _taggedTemplateLiteral2["default"])(["\n  height: ", ";\n  width: ", ";\n  object-fit: ", ";\n"])), function (_ref11) {
+var TagImage = _styledComponents["default"].img(_templateObject3 || (_templateObject3 = (0, _taggedTemplateLiteral2["default"])(["\n  height: ", ";\n  width: ", ";\n  object-fit: ", ";\n"])), function (_ref11) {
   var imagePadding = _ref11.imagePadding;
-  return !imagePadding ? "100%" : "calc(100% - ".concat(_variables.spaces[imagePadding], " - ").concat(_variables.spaces[imagePadding], ")");
+  return !imagePadding ? "100%" : (0, _typeof2["default"])(imagePadding) !== "object" && "calc(100% - ".concat(_variables.spaces[imagePadding], " - ").concat(_variables.spaces[imagePadding], ")");
 }, function (_ref12) {
   var imagePadding = _ref12.imagePadding;
-  return !imagePadding ? "100%" : "calc(100% - ".concat(_variables.spaces[imagePadding], " - ").concat(_variables.spaces[imagePadding], ")");
+  return !imagePadding ? "100%" : (0, _typeof2["default"])(imagePadding) !== "object" && "calc(100% - ".concat(_variables.spaces[imagePadding], " - ").concat(_variables.spaces[imagePadding], ")");
 }, function (_ref13) {
-  var cover = _ref13.cover;
-  return cover ? "cover" : "contain";
+  var imageCover = _ref13.imageCover;
+  return imageCover ? "cover" : "contain";
 });
-
-var ImageContainer = _styledComponents["default"].div(_templateObject5 || (_templateObject5 = (0, _taggedTemplateLiteral2["default"])(["\n  width: 35%;\n  height: 100%;\n  flex-shrink: 0;\n  background: ", ";\n  justify-content: center;\n  align-items: center;\n  display: inline-flex;\n"])), function (_ref14) {
+var ImageContainer = _styledComponents["default"].div(_templateObject4 || (_templateObject4 = (0, _taggedTemplateLiteral2["default"])(["\n  display: inline-flex;\n  justify-content: center;\n  align-items: center;\n  flex-shrink: 0;\n  width: 35%;\n  height: 100%;\n  background-color: ", ";\n"])), function (_ref14) {
   var imageBgColor = _ref14.imageBgColor;
   return imageBgColor;
 });
-
-var CardContent = _styledComponents["default"].div(_templateObject6 || (_templateObject6 = (0, _taggedTemplateLiteral2["default"])(["\n  flex-grow: 1;\n  padding: ", ";\n  padding-top: ", ";\n  padding-right: ", ";\n  padding-bottom: ", ";\n  padding-left: ", ";\n  overflow: hidden;\n"])), function (_ref15) {
-  var contentPadding = _ref15.contentPadding;
-  return contentPadding && (0, _typeof2["default"])(contentPadding) !== "object" ? _variables.spaces[contentPadding] : "0px";
-}, function (_ref16) {
-  var contentPadding = _ref16.contentPadding;
-  return contentPadding && (0, _typeof2["default"])(contentPadding) === "object" && contentPadding.top ? _variables.spaces[contentPadding.top] : "";
-}, function (_ref17) {
-  var contentPadding = _ref17.contentPadding;
-  return contentPadding && (0, _typeof2["default"])(contentPadding) === "object" && contentPadding.right ? _variables.spaces[contentPadding.right] : "";
-}, function (_ref18) {
-  var contentPadding = _ref18.contentPadding;
-  return contentPadding && (0, _typeof2["default"])(contentPadding) === "object" && contentPadding.bottom ? _variables.spaces[contentPadding.bottom] : "";
-}, function (_ref19) {
-  var contentPadding = _ref19.contentPadding;
-  return contentPadding && (0, _typeof2["default"])(contentPadding) === "object" && contentPadding.left ? _variables.spaces[contentPadding.left] : "";
-});
-
-var _default = DxcCard;
-exports["default"] = _default;
+var CardContent = _styledComponents["default"].div(_templateObject5 || (_templateObject5 = (0, _taggedTemplateLiteral2["default"])(["\n  flex-grow: 1;\n  overflow: hidden;\n"])));
+var _default = exports["default"] = DxcCard;

package/card/Card.stories.tsx

@@ -2,7 +2,6 @@ import React from "react";
 import Title from "../../.storybook/components/Title";
 import ExampleContainer from "../../.storybook/components/ExampleContainer";
 import DxcCard from "./Card";
-import imagePath from "./ice-cream.jpg";
 import { userEvent, within } from "@storybook/testing-library";
 
 export default {
@@ -38,104 +37,75 @@ const Card = () => (
     </ExampleContainer>
     <Title title="Default with image" theme="light" level={4} />
     <ExampleContainer>
-      <DxcCard imageSrc={imagePath}>Default</DxcCard>
+      <DxcCard imageSrc="https://picsum.photos/id/1022/200/300">Default</DxcCard>
     </ExampleContainer>
     <Title title="Default image with background color" theme="light" level={4} />
     <ExampleContainer>
-      <DxcCard imageSrc={imagePath} imageBgColor="yellow">
+      <DxcCard imageSrc="https://picsum.photos/id/1022/200/300" imageBgColor="yellow">
         Background color
       </DxcCard>
     </ExampleContainer>
     <Title title="Default image with position after" theme="light" level={4} />
     <ExampleContainer>
-      <DxcCard imageSrc={imagePath} imagePosition="after">
+      <DxcCard imageSrc="https://picsum.photos/id/1022/200/300" imagePosition="after">
         Position after
       </DxcCard>
     </ExampleContainer>
     <Title title="Image cover" theme="light" level={4} />
     <ExampleContainer>
-      <DxcCard imageSrc={imagePath} imageCover>
+      <DxcCard imageSrc="https://picsum.photos/id/1022/200/300" imageCover>
         Image cover
       </DxcCard>
     </ExampleContainer>
     <Title title="Image cover with position after" theme="light" level={4} />
     <ExampleContainer>
-      <DxcCard imageSrc={imagePath} imageCover imagePosition="after">
+      <DxcCard imageSrc="https://picsum.photos/id/1022/200/300" imageCover imagePosition="after">
         Image cover with position after
       </DxcCard>
     </ExampleContainer>
     <Title title="Image padding" theme="light" level={2} />
     <ExampleContainer>
       <Title title="Xxsmall" theme="light" level={4} />
-      <DxcCard imageSrc={imagePath} imagePadding="xxsmall" imageCover>
+      <DxcCard imageSrc="https://picsum.photos/id/1022/200/300" imagePadding="xxsmall" imageCover>
         Xxsmall
       </DxcCard>
     </ExampleContainer>
     <ExampleContainer>
       <Title title="Xsmall" theme="light" level={4} />
-      <DxcCard imageSrc={imagePath} imagePadding="xsmall" imageCover>
+      <DxcCard imageSrc="https://picsum.photos/id/1022/200/300" imagePadding="xsmall" imageCover>
         Xsmall
       </DxcCard>
     </ExampleContainer>
     <ExampleContainer>
       <Title title="Small" theme="light" level={4} />
-      <DxcCard imageSrc={imagePath} imagePadding="small" imageCover>
+      <DxcCard imageSrc="https://picsum.photos/id/1022/200/300" imagePadding="small" imageCover>
         Small
       </DxcCard>
     </ExampleContainer>
     <ExampleContainer>
       <Title title="Medium" theme="light" level={4} />
-      <DxcCard imageSrc={imagePath} imagePadding="medium" imageCover>
+      <DxcCard imageSrc="https://picsum.photos/id/1022/200/300" imagePadding="medium" imageCover>
         Medium
       </DxcCard>
     </ExampleContainer>
     <ExampleContainer>
       <Title title="Large" theme="light" level={4} />
-      <DxcCard imageSrc={imagePath} imagePadding="large" imageCover>
+      <DxcCard imageSrc="https://picsum.photos/id/1022/200/300" imagePadding="large" imageCover>
         Large
       </DxcCard>
     </ExampleContainer>
     <ExampleContainer>
       <Title title="Xlarge" theme="light" level={4} />
-      <DxcCard imageSrc={imagePath} imagePadding="xlarge" imageCover>
+      <DxcCard imageSrc="https://picsum.photos/id/1022/200/300" imagePadding="xlarge" imageCover>
         Xlarge
       </DxcCard>
     </ExampleContainer>
     <ExampleContainer>
       <Title title="Xxlarge" theme="light" level={4} />
-      <DxcCard imageSrc={imagePath} imagePadding="xxlarge" imageCover>
+      <DxcCard imageSrc="https://picsum.photos/id/1022/200/300" imagePadding="xxlarge" imageCover>
         Xxlarge
       </DxcCard>
     </ExampleContainer>
-    <Title title="Content padding" theme="light" level={2} />
-    <ExampleContainer>
-      <Title title="Xxsmall" theme="light" level={4} />
-      <DxcCard contentPadding="xxsmall">Xxsmall</DxcCard>
-    </ExampleContainer>
-    <ExampleContainer>
-      <Title title="Xsmall" theme="light" level={4} />
-      <DxcCard contentPadding="xsmall">Xsmall</DxcCard>
-    </ExampleContainer>
-    <ExampleContainer>
-      <Title title="Small" theme="light" level={4} />
-      <DxcCard contentPadding="small">Small</DxcCard>
-    </ExampleContainer>
-    <ExampleContainer>
-      <Title title="Medium" theme="light" level={4} />
-      <DxcCard contentPadding="medium">Medium</DxcCard>
-    </ExampleContainer>
-    <ExampleContainer>
-      <Title title="Large" theme="light" level={4} />
-      <DxcCard contentPadding="large">Large</DxcCard>
-    </ExampleContainer>
-    <ExampleContainer>
-      <Title title="Xlarge" theme="light" level={4} />
-      <DxcCard contentPadding="xlarge">Xlarge</DxcCard>
-    </ExampleContainer>
-    <ExampleContainer>
-      <Title title="Xxlarge" theme="light" level={4} />
-      <DxcCard contentPadding="xxlarge">Xxlarge</DxcCard>
-    </ExampleContainer>
     <Title title="Margin" theme="light" level={2} />
     <ExampleContainer>
       <Title title="Xxsmall" theme="light" level={4} />

package/card/Card.test.js

@@ -1,50 +1,39 @@
 "use strict";
 
 var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault");
-
 var _react = _interopRequireDefault(require("react"));
-
 var _react2 = require("@testing-library/react");
-
-var _Card = _interopRequireDefault(require("./Card"));
-
+var _Card = _interopRequireDefault(require("./Card.tsx"));
 describe("Card component tests", function () {
   test("Card renders with correct content", function () {
     var _render = (0, _react2.render)( /*#__PURE__*/_react["default"].createElement(_Card["default"], null, "test-card")),
-        getByText = _render.getByText;
-
+      getByText = _render.getByText;
     expect(getByText("test-card")).toBeTruthy();
   });
   test("Card renders with correct href", function () {
     var _render2 = (0, _react2.render)( /*#__PURE__*/_react["default"].createElement(_Card["default"], {
-      linkHref: "/testPage"
-    }, "test-card")),
-        getByRole = _render2.getByRole;
-
+        linkHref: "/testPage"
+      }, "test-card")),
+      getByRole = _render2.getByRole;
     var card = getByRole("link");
     expect(card.getAttribute("href")).toEqual("/testPage");
   });
   test("Card renders with correct image", function () {
     var _render3 = (0, _react2.render)( /*#__PURE__*/_react["default"].createElement(_Card["default"], {
-      imageSrc: "/testImage"
-    }, "test-card")),
-        getByRole = _render3.getByRole;
-
+        imageSrc: "/testImage"
+      }, "test-card")),
+      getByRole = _render3.getByRole;
     var card = getByRole("img");
     expect(card.getAttribute("src")).toEqual("/testImage");
   });
   test("OnClick function is called", function () {
     var onClick = jest.fn();
-
     var _render4 = (0, _react2.render)( /*#__PURE__*/_react["default"].createElement(_Card["default"], {
-      onClick: onClick
-    }, "test-card")),
-        getByText = _render4.getByText;
-
+        onClick: onClick
+      }, "test-card")),
+      getByText = _render4.getByText;
     var card = getByText("test-card");
-
     _react2.fireEvent.click(card);
-
     expect(onClick).toHaveBeenCalled();
   });
 });

package/card/types.d.ts

@@ -1,12 +1,12 @@
 /// <reference types="react" />
-declare type Space = "xxsmall" | "xsmall" | "small" | "medium" | "large" | "xlarge" | "xxlarge";
-declare type Size = {
+type Space = "xxsmall" | "xsmall" | "small" | "medium" | "large" | "xlarge" | "xxlarge";
+type Size = {
     top?: Space;
     bottom?: Space;
     left?: Space;
     right?: Space;
 };
-declare type Props = {
+type Props = {
     /**
      * URL of the image that will be placed in the card component.
      * In case of omission, the image container will not appear and
@@ -47,20 +47,15 @@ declare type Props = {
      */
     margin?: Space | Size;
     /**
-     * Size of the padding to be applied to the content area ('xxsmall' | 'xsmall' | 'small' | 'medium' | 'large' | 'xlarge' | 'xxlarge').
-     * You can pass an object with 'top', 'bottom', 'left' and 'right' properties in order to specify different padding sizes.
-     */
-    contentPadding?: Space | Size;
-    /**
-     * Value of the tabindex given when there is an href.
+     * Value of the tabindex attribute applied when the component is clickable.
      */
     tabIndex?: number;
     /**
-     * Whether the card must be outlined.
+     * Determines whether or not the component should have an outline.
      */
     outlined?: boolean;
     /**
-     * Custom content that will be placed in the card component.
+     * Custom content that will be placed inside the component.
      */
     children?: React.ReactNode;
 };