@dxc-technology/halstack-react 0.0.0-b4fde6b → 0.0.0-b5ec444

package/card/Card.js

@@ -1,70 +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");
-
 var _useTheme = _interopRequireDefault(require("../useTheme"));
-
 var _Box = _interopRequireDefault(require("../box/Box"));
-
 var _templateObject, _templateObject2, _templateObject3, _templateObject4, _templateObject5;
-
-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; }
-
+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 && Object.prototype.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 imageComponent = /*#__PURE__*/_react["default"].createElement(ImageContainer, {
-    imageBgColor: imageBgColor
-  }, /*#__PURE__*/_react["default"].createElement(TagImage, {
-    imagePadding: imagePadding,
-    imageCover: imageCover,
-    src: imageSrc
-  }));
-
-  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);
@@ -73,7 +49,6 @@ var DxcCard = function DxcCard(_ref) {
       return changeIsHovered(false);
     },
     onClick: onClick,
-    hasAction: onClick || linkHref,
     tabIndex: onClick || linkHref ? tabIndex : -1,
     as: linkHref && "a",
     href: linkHref
@@ -82,13 +57,17 @@ var DxcCard = function DxcCard(_ref) {
   }, /*#__PURE__*/_react["default"].createElement(_styledComponents.ThemeProvider, {
     theme: colorsTheme.card
   }, /*#__PURE__*/_react["default"].createElement(CardContainer, {
-    hasAction: onClick || linkHref
-  }, imageSrc && imagePosition === "before" && imageComponent, /*#__PURE__*/_react["default"].createElement(CardContent, {
-    contentPadding: contentPadding
-  }, children), imageSrc && imagePosition === "after" && imageComponent))));
+    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
+  })), /*#__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  text-decoration: none;\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) {
@@ -111,10 +90,11 @@ var StyledDxcCard = _styledComponents["default"].div(_templateObject || (_templa
   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  }";
+  return hasAction && ":focus {\n      outline: #0095ff auto 1px;\n    }";
 });
-
-var CardContainer = _styledComponents["default"].div(_templateObject2 || (_templateObject2 = (0, _taggedTemplateLiteral2["default"])(["\n  display: inline-flex;\n  height: ", ";\n  width: ", ";\n  &:hover {\n    border-color: ", ";\n  }\n"])), function (props) {
+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) {
   return props.theme.width;
@@ -122,7 +102,6 @@ var CardContainer = _styledComponents["default"].div(_templateObject2 || (_templ
   var hasAction = _ref10.hasAction;
   return hasAction ? "" : "unset";
 });
-
 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%" : (0, _typeof2["default"])(imagePadding) !== "object" && "calc(100% - ".concat(_variables.spaces[imagePadding], " - ").concat(_variables.spaces[imagePadding], ")");
@@ -133,28 +112,9 @@ var TagImage = _styledComponents["default"].img(_templateObject3 || (_templateOb
   var imageCover = _ref13.imageCover;
   return imageCover ? "cover" : "contain";
 });
-
-var ImageContainer = _styledComponents["default"].div(_templateObject4 || (_templateObject4 = (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(_templateObject5 || (_templateObject5 = (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

@@ -106,35 +106,6 @@ const Card = () => (
         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.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,21 +47,15 @@ declare type Props = {
      */
     margin?: Space | Size;
     /**
-     * @deprecated This prop will be removed shortly, consider using the Inset component for this purpose.
-     * 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;
 };