@dxc-technology/halstack-react 6.2.2 → 7.0.0

package/accordion/types.d.ts

@@ -56,6 +56,7 @@ declare type Props = {
      */
     margin?: Space | Margin;
     /**
+     * @deprecated This prop will be removed shortly, consider using the Inset component for this purpose.
      * Size of the padding to be applied to the custom 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.
      */

package/accordion-group/types.d.ts

@@ -31,6 +31,7 @@ export declare type AccordionPropsType = {
      */
     disabled?: boolean;
     /**
+     * @deprecated This prop will be removed shortly, consider using the Inset component for this purpose.
      * Size of the padding to be applied to the custom 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.
      */

package/box/types.d.ts

@@ -31,6 +31,7 @@ declare type Props = {
      */
     margin?: Space | Margin;
     /**
+     * @deprecated This prop will be removed shortly, consider using the Inset component for this purpose.
      * Size of the padding to be applied to the custom 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.
      */

package/card/types.d.ts

@@ -47,6 +47,7 @@ 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.
      */

package/common/variables.js

@@ -763,13 +763,13 @@ var componentTokens = {
     fontTextTransform: "none",
     verticalPadding: "0.75rem",
     horizontalPadding: "2rem",
-    marginRight: "2rem",
-    marginLeft: "2rem",
+    marginRight: "40px",
+    marginLeft: "20px",
     itemsPerPageSelectorMarginLeft: "0px",
-    itemsPerPageSelectorMarginRight: "1rem",
+    itemsPerPageSelectorMarginRight: "0.5rem",
     pageSelectorMarginRight: "30px",
     pageSelectorMarginLeft: "0px",
-    totalItemsContainerMarginRight: "3rem",
+    totalItemsContainerMarginRight: "2.5rem",
     totalItemsContainerMarginLeft: "0px"
   },
   progressBar: {
@@ -1462,7 +1462,7 @@ var defaultTranslatedComponentLabels = {
     incrementValueTitle: "Increment value"
   },
   paginator: {
-    itemsPerPageText: "Items per page ",
+    itemsPerPageText: "Items per page: ",
     minToMaxOfText: function minToMaxOfText(minNumberOfItems, maxNumberOfItems, totalItems) {
       return "".concat(minNumberOfItems, " to ").concat(maxNumberOfItems, " of ").concat(totalItems);
     },

package/dialog/types.d.ts

@@ -26,6 +26,7 @@ declare type Props = {
      */
     onBackgroundClick?: () => void;
     /**
+     * @deprecated This prop will be removed shortly, consider using the Inset component for this purpose.
      * Size of the padding to be applied to the component ('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.
      */

package/dropdown/Dropdown.js

@@ -254,7 +254,8 @@ var DxcDropdown = function DxcDropdown(_ref) {
   }, /*#__PURE__*/_react["default"].createElement(Popover.Root, {
     open: isOpen
   }, /*#__PURE__*/_react["default"].createElement(Popover.Trigger, {
-    asChild: true
+    asChild: true,
+    type: undefined
   }, /*#__PURE__*/_react["default"].createElement(DropdownTrigger, {
     opened: isOpen,
     onClick: handleTriggerOnClick,
@@ -267,8 +268,8 @@ var DxcDropdown = function DxcDropdown(_ref) {
     margin: margin,
     size: size,
     id: triggerId,
-    "aria-disabled": disabled,
     "aria-haspopup": "true",
+    "aria-controls": menuId,
     "aria-expanded": isOpen ? true : undefined,
     tabIndex: tabIndex,
     ref: triggerRef

package/dropdown/Dropdown.test.js

@@ -74,7 +74,6 @@ describe("Dropdown component tests", function () {
         getByRole = _render.getByRole;
 
     var dropdown = getByRole("button");
-    expect(dropdown.getAttribute("aria-disabled")).toBe("false");
     expect(dropdown.getAttribute("aria-haspopup")).toBe("true");
     expect(dropdown.getAttribute("aria-expanded")).toBeNull();
     expect(dropdown.getAttribute("aria-activedescendant")).toBeNull();
@@ -82,6 +81,7 @@ describe("Dropdown component tests", function () {
     _userEvent["default"].click(dropdown);
 
     var menu = getByRole("menu");
+    expect(dropdown.getAttribute("aria-controls")).toBe(menu.id);
     expect(dropdown.getAttribute("aria-expanded")).toBe("true");
     expect(menu.getAttribute("aria-activedescendant")).toBe("option-0");
     expect(menu.getAttribute("aria-orientation")).toBe("vertical");
@@ -123,7 +123,6 @@ describe("Dropdown component tests", function () {
         queryByText = _render3.queryByText;
 
     var dropdown = getByRole("button");
-    expect(dropdown.getAttribute("aria-disabled")).toBe("true");
     expect(queryByRole("menu")).toBeFalsy();
 
     _userEvent["default"].click(dropdown);

package/file-input/FileInput.js

@@ -405,12 +405,16 @@ var DxcFileInput = function DxcFileInput(_ref) {
   }, helperText), mode === "file" ? /*#__PURE__*/_react["default"].createElement(FileContainer, {
     multiple: multiple,
     files: files
-  }, /*#__PURE__*/_react["default"].createElement(HiddenInputFile, {
+  }, /*#__PURE__*/_react["default"].createElement(ValueInput, {
     id: fileInputId,
     type: "file",
     accept: accept,
     multiple: multiple,
-    onChange: selectFiles
+    onChange: selectFiles,
+    name: name,
+    disabled: disabled,
+    readOnly: true,
+    "aria-hidden": "true"
   }), /*#__PURE__*/_react["default"].createElement(_Button["default"], {
     mode: "secondary",
     label: buttonLabel !== null && buttonLabel !== void 0 ? buttonLabel : multiple ? translatedLabels.fileInput.multipleButtonLabelDefault : translatedLabels.fileInput.singleButtonLabelDefault,
@@ -435,12 +439,16 @@ var DxcFileInput = function DxcFileInput(_ref) {
       onDelete: onDelete,
       tabIndex: tabIndex
     })));
-  })) : /*#__PURE__*/_react["default"].createElement(Container, null, /*#__PURE__*/_react["default"].createElement(HiddenInputFile, {
+  })) : /*#__PURE__*/_react["default"].createElement(Container, null, /*#__PURE__*/_react["default"].createElement(ValueInput, {
     id: fileInputId,
     type: "file",
     accept: accept,
     multiple: multiple,
-    onChange: selectFiles
+    onChange: selectFiles,
+    name: name,
+    disabled: disabled,
+    readOnly: true,
+    "aria-hidden": "true"
   }), /*#__PURE__*/_react["default"].createElement(DragDropArea, {
     isDragging: isDragging,
     disabled: disabled,
@@ -543,7 +551,7 @@ var FileContainer = _styledComponents["default"].div(_templateObject5 || (_templ
   return props.multiple || props.files.length > 1 ? "column" : "row";
 });
 
-var HiddenInputFile = _styledComponents["default"].input(_templateObject6 || (_templateObject6 = (0, _taggedTemplateLiteral2["default"])(["\n  visibility: hidden;\n  position: absolute;\n  width: 0px;\n  height: 0px;\n"])));
+var ValueInput = _styledComponents["default"].input(_templateObject6 || (_templateObject6 = (0, _taggedTemplateLiteral2["default"])(["\n  display: none;\n"])));
 
 var ButtonContainer = _styledComponents["default"].div(_templateObject7 || (_templateObject7 = (0, _taggedTemplateLiteral2["default"])(["\n  ", ";\n"])), function (props) {
   return props.mode === "filedrop" && "padding: 2px 12px 2px 3px";

package/file-input/FileInput.test.js

@@ -454,4 +454,45 @@ describe("FileInput component tests", function () {
       }
     }, _callee6);
   })));
+  test("File input sends value when submitted in a form", function () {
+    var newFile = new File(["newFile"], "newFile.pdf", {
+      type: "pdf"
+    });
+    var handlerOnSubmit = jest.fn(function (e) {
+      e.preventDefault();
+      var formData = new FormData(e.target);
+      var formProps = Object.fromEntries(formData);
+      expect(formProps).toStrictEqual({
+        file: newFile
+      });
+    });
+
+    var _render17 = (0, _react2.render)( /*#__PURE__*/_react["default"].createElement("form", {
+      onSubmit: handlerOnSubmit
+    }, /*#__PURE__*/_react["default"].createElement(_FileInput["default"], {
+      label: "File input label",
+      name: "file",
+      helperText: "File input helper text",
+      mode: "filedrop",
+      buttonLabel: "Choose",
+      dropAreaLabel: "(or drop the files)"
+    }), /*#__PURE__*/_react["default"].createElement("button", {
+      type: "submit"
+    }, "Submit"))),
+        getByText = _render17.getByText,
+        getByLabelText = _render17.getByLabelText;
+
+    var inputFile = getByLabelText("File input label", {
+      hidden: true
+    });
+    var submit = getByText("Submit");
+
+    _react2.fireEvent.change(inputFile, {
+      target: {
+        files: [newFile]
+      }
+    });
+
+    _userEvent["default"].click(submit);
+  });
 });

package/flex/Flex.d.ts

@@ -1,4 +1,4 @@
 /// <reference types="react" />
 import FlexPropsType from "./types";
-declare const DxcFlex: ({ children, ...props }: FlexPropsType) => JSX.Element;
+declare const DxcFlex: ({ direction, wrap, gap, order, grow, shrink, basis, children, ...props }: FlexPropsType) => JSX.Element;
 export default DxcFlex;

package/flex/Flex.js

@@ -11,6 +11,8 @@ var _typeof2 = _interopRequireDefault(require("@babel/runtime/helpers/typeof"));
 
 var _taggedTemplateLiteral2 = _interopRequireDefault(require("@babel/runtime/helpers/taggedTemplateLiteral"));
 
+var _extends2 = _interopRequireDefault(require("@babel/runtime/helpers/extends"));
+
 var _objectWithoutProperties2 = _interopRequireDefault(require("@babel/runtime/helpers/objectWithoutProperties"));
 
 var _react = _interopRequireDefault(require("react"));
@@ -19,20 +21,39 @@ var _styledComponents = _interopRequireDefault(require("styled-components"));
 
 var _templateObject;
 
-var _excluded = ["children"];
+var _excluded = ["direction", "wrap", "gap", "order", "grow", "shrink", "basis", "children"],
+    _excluded2 = ["justifyContent", "alignItems", "alignContent", "alignSelf"];
 
 var DxcFlex = function DxcFlex(_ref) {
-  var children = _ref.children,
+  var _ref$direction = _ref.direction,
+      direction = _ref$direction === void 0 ? "row" : _ref$direction,
+      _ref$wrap = _ref.wrap,
+      wrap = _ref$wrap === void 0 ? "nowrap" : _ref$wrap,
+      _ref$gap = _ref.gap,
+      gap = _ref$gap === void 0 ? "0" : _ref$gap,
+      _ref$order = _ref.order,
+      order = _ref$order === void 0 ? 0 : _ref$order,
+      _ref$grow = _ref.grow,
+      grow = _ref$grow === void 0 ? 0 : _ref$grow,
+      _ref$shrink = _ref.shrink,
+      shrink = _ref$shrink === void 0 ? 1 : _ref$shrink,
+      _ref$basis = _ref.basis,
+      basis = _ref$basis === void 0 ? "auto" : _ref$basis,
+      children = _ref.children,
       props = (0, _objectWithoutProperties2["default"])(_ref, _excluded);
-  return /*#__PURE__*/_react["default"].createElement(Flex, props, children);
+  return /*#__PURE__*/_react["default"].createElement(Flex, (0, _extends2["default"])({
+    $direction: direction,
+    $wrap: wrap,
+    $order: order,
+    $grow: grow,
+    $shrink: shrink,
+    $basis: basis,
+    $gap: gap
+  }, props), children);
 };
 
 var Flex = _styledComponents["default"].div(_templateObject || (_templateObject = (0, _taggedTemplateLiteral2["default"])(["\n  display: flex;\n  ", "\n"])), function (_ref2) {
-  var _ref2$direction = _ref2.direction,
-      direction = _ref2$direction === void 0 ? "row" : _ref2$direction,
-      _ref2$wrap = _ref2.wrap,
-      wrap = _ref2$wrap === void 0 ? "nowrap" : _ref2$wrap,
-      _ref2$justifyContent = _ref2.justifyContent,
+  var _ref2$justifyContent = _ref2.justifyContent,
       justifyContent = _ref2$justifyContent === void 0 ? "flex-start" : _ref2$justifyContent,
       _ref2$alignItems = _ref2.alignItems,
       alignItems = _ref2$alignItems === void 0 ? "stretch" : _ref2$alignItems,
@@ -40,17 +61,8 @@ var Flex = _styledComponents["default"].div(_templateObject || (_templateObject
       alignContent = _ref2$alignContent === void 0 ? "normal" : _ref2$alignContent,
       _ref2$alignSelf = _ref2.alignSelf,
       alignSelf = _ref2$alignSelf === void 0 ? "auto" : _ref2$alignSelf,
-      _ref2$gap = _ref2.gap,
-      gap = _ref2$gap === void 0 ? "0" : _ref2$gap,
-      _ref2$order = _ref2.order,
-      order = _ref2$order === void 0 ? 0 : _ref2$order,
-      _ref2$grow = _ref2.grow,
-      grow = _ref2$grow === void 0 ? 0 : _ref2$grow,
-      _ref2$shrink = _ref2.shrink,
-      shrink = _ref2$shrink === void 0 ? 1 : _ref2$shrink,
-      _ref2$basis = _ref2.basis,
-      basis = _ref2$basis === void 0 ? "auto" : _ref2$basis;
-  return "\n    flex-direction: ".concat(direction, "; \n    flex-wrap: ").concat(wrap, "; \n    justify-content: ").concat(justifyContent, "; \n    align-items: ").concat(alignItems, ";\n    align-content: ").concat(alignContent, ";\n    align-self: ").concat(alignSelf, ";\n    gap: ").concat((0, _typeof2["default"])(gap) === "object" ? "".concat(gap.rowGap, " ").concat(gap.columnGap) : gap, ";\n    order: ").concat(order, ";\n    flex-grow: ").concat(grow, ";\n    flex-shrink: ").concat(shrink, ";\n    flex-basis: ").concat(basis, ";\n  ");
+      props = (0, _objectWithoutProperties2["default"])(_ref2, _excluded2);
+  return "\n    flex-direction: ".concat(props.$direction, "; \n    flex-wrap: ").concat(props.$wrap, "; \n    justify-content: ").concat(justifyContent, "; \n    align-items: ").concat(alignItems, ";\n    align-content: ").concat(alignContent, ";\n    align-self: ").concat(alignSelf, ";\n    gap: ").concat((0, _typeof2["default"])(props.$gap) === "object" ? "".concat(props.$gap.rowGap, " ").concat(props.$gap.columnGap) : props.$gap, ";\n    order: ").concat(props.$order, ";\n    flex-grow: ").concat(props.$grow, ";\n    flex-shrink: ").concat(props.$shrink, ";\n    flex-basis: ").concat(props.$basis, ";\n  ");
 });
 
 var _default = DxcFlex;

package/flex/types.d.ts

@@ -3,13 +3,15 @@ declare type Gap = {
     rowGap: string;
     columnGap: string;
 };
-declare type Props = {
-    direction?: "row" | "row-reverse" | "column" | "column-reverse";
-    wrap?: "nowrap" | "wrap" | "wrap-reverse";
+declare type CommonProps = {
     justifyContent?: "flex-start" | "flex-end" | "start" | "end" | "left" | "right" | "center" | "space-between" | "space-around" | "space-evenly";
     alignItems?: "stretch" | "flex-start" | "flex-end" | "start" | "end" | "self-start" | "self-end" | "center" | "baseline";
     alignContent?: "normal" | "flex-start" | "flex-end" | "start" | "end" | "center" | "space-between" | "space-around" | "space-evenly" | "stretch";
     alignSelf?: "auto" | "flex-start" | "flex-end" | "center" | "baseline" | "stretch";
+};
+declare type Props = CommonProps & {
+    direction?: "row" | "row-reverse" | "column" | "column-reverse";
+    wrap?: "nowrap" | "wrap" | "wrap-reverse";
     gap?: string | Gap;
     order?: number;
     grow?: number;
@@ -18,4 +20,13 @@ declare type Props = {
     as?: keyof HTMLElementTagNameMap;
     children: React.ReactNode;
 };
+export declare type StyledProps = CommonProps & {
+    $direction?: "row" | "row-reverse" | "column" | "column-reverse";
+    $wrap?: "nowrap" | "wrap" | "wrap-reverse";
+    $gap?: string | Gap;
+    $order?: number;
+    $grow?: number;
+    $shrink?: number;
+    $basis?: string;
+};
 export default Props;

package/footer/types.d.ts

@@ -57,6 +57,7 @@ declare type FooterPropsType = {
      */
     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 custom area of the component.
      * You can pass an object with properties in order to specify different padding sizes.
      */

package/header/types.d.ts

@@ -32,6 +32,7 @@ declare type Props = {
      */
     margin?: Space;
     /**
+     * @deprecated This prop will be removed shortly, consider using the Inset component for this purpose.
      * Size of the padding to be applied to the custom area of the component
      * ('xxsmall' | 'xsmall' | 'small' | 'medium' | 'large' | 'xlarge' | 'xxlarge').
      * You can pass an object with 'top', 'bottom', 'left' and 'right' properties in

package/number-input/NumberInput.test.js

@@ -83,7 +83,7 @@ describe("Number input component tests", function () {
     })),
         getByRole = _render8.getByRole;
 
-    var input = getByRole("textbox");
+    var input = getByRole("spinbutton");
 
     _userEvent["default"].type(input, "1");
 
@@ -503,4 +503,41 @@ describe("Number input component tests", function () {
     var increment = getAllByRole("button")[1];
     expect(increment.getAttribute("aria-label")).toBe("Increment value");
   });
+  test("Number input submits correct values in a form", function () {
+    var handlerOnSubmit = jest.fn(function (e) {
+      e.preventDefault();
+      var formData = new FormData(e.target);
+      var formProps = Object.fromEntries(formData);
+      expect(formProps).toStrictEqual({
+        data: "0"
+      });
+    });
+
+    var _render23 = (0, _react2.render)( /*#__PURE__*/_react["default"].createElement("form", {
+      onSubmit: handlerOnSubmit
+    }, /*#__PURE__*/_react["default"].createElement(_NumberInput["default"], {
+      label: "Number input label",
+      name: "data"
+    }), /*#__PURE__*/_react["default"].createElement("button", {
+      type: "submit"
+    }, "Submit"))),
+        getByText = _render23.getByText,
+        getAllByRole = _render23.getAllByRole;
+
+    var less = getAllByRole("button")[0];
+    var more = getAllByRole("button")[1];
+    var submit = getByText("Submit");
+
+    _userEvent["default"].click(more);
+
+    expect(handlerOnSubmit).not.toHaveBeenCalled();
+
+    _userEvent["default"].click(less);
+
+    expect(handlerOnSubmit).not.toHaveBeenCalled();
+
+    _userEvent["default"].click(submit);
+
+    expect(handlerOnSubmit).toHaveBeenCalled();
+  });
 });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dxc-technology/halstack-react",
-  "version": "6.2.2",
+  "version": "7.0.0",
   "description": "DXC Halstack React components library",
   "repository": "dxc-technology/halstack-react",
   "homepage": "https://developer.dxc.com/halstack",

package/paginator/Paginator.js

@@ -153,7 +153,7 @@ var ItemsPageContainer = _styledComponents["default"].span(_templateObject3 || (
 
 var ItemsLabel = _styledComponents["default"].span(_templateObject4 || (_templateObject4 = (0, _taggedTemplateLiteral2["default"])(["\n  margin-right: 0.5rem;\n"])));
 
-var GoToLabel = _styledComponents["default"].span(_templateObject5 || (_templateObject5 = (0, _taggedTemplateLiteral2["default"])(["\n  margin-right: 0.5rem;\n  margin-left: 1rem;\n"])));
+var GoToLabel = _styledComponents["default"].span(_templateObject5 || (_templateObject5 = (0, _taggedTemplateLiteral2["default"])(["\n  margin-right: 0.5rem;\n  margin-left: 0.5rem;\n"])));
 
 var TotalItemsContainer = _styledComponents["default"].span(_templateObject6 || (_templateObject6 = (0, _taggedTemplateLiteral2["default"])(["\n  margin-right: ", ";\n  margin-left: ", ";\n"])), function (props) {
   return props.theme.totalItemsContainerMarginRight;
@@ -163,7 +163,7 @@ var TotalItemsContainer = _styledComponents["default"].span(_templateObject6 ||
 
 var LabelsContainer = _styledComponents["default"].div(_templateObject7 || (_templateObject7 = (0, _taggedTemplateLiteral2["default"])(["\n  display: flex;\n  gap: 0.5rem;\n  width: 100%;\n  justify-content: flex-end;\n  align-items: center;\n"])));
 
-var PageToSelectContainer = _styledComponents["default"].span(_templateObject8 || (_templateObject8 = (0, _taggedTemplateLiteral2["default"])(["\n  display: flex;\n  align-items: center;\n\n  label {\n    height: 0px;\n  }\n  label + .MuiInput-formControl {\n    margin-top: 0px;\n  }\n"])));
+var PageToSelectContainer = _styledComponents["default"].span(_templateObject8 || (_templateObject8 = (0, _taggedTemplateLiteral2["default"])(["\n  display: flex;\n  align-items: center;\n  margin-right: 0.5rem;\n\n  label {\n    height: 0px;\n  }\n  label + .MuiInput-formControl {\n    margin-top: 0px;\n  }\n"])));
 
 var TextContainer = _styledComponents["default"].span(_templateObject9 || (_templateObject9 = (0, _taggedTemplateLiteral2["default"])([""])));
 

package/paginator/Paginator.test.js

@@ -77,7 +77,7 @@ describe("Paginator component tests", function () {
     })),
         getByText = _render3.getByText;
 
-    expect(getByText("Items per page")).toBeTruthy();
+    expect(getByText("Items per page:")).toBeTruthy();
   });
   test("Paginator renders with itemsPerPageOptions", function () {
     var _render4 = (0, _react2.render)( /*#__PURE__*/_react["default"].createElement(_Paginator["default"], {

package/password-input/PasswordInput.test.js

@@ -46,9 +46,9 @@ describe("Password input component tests", function () {
       label: "Password input",
       onChange: onChange
     })),
-        getByRole = _render4.getByRole;
+        getByLabelText = _render4.getByLabelText;
 
-    var passwordInput = getByRole("textbox");
+    var passwordInput = getByLabelText("Password input");
 
     _userEvent["default"].type(passwordInput, "Pa$$w0rd");
 
@@ -64,9 +64,9 @@ describe("Password input component tests", function () {
       label: "Password input",
       onBlur: onBlur
     })),
-        getByRole = _render5.getByRole;
+        getByLabelText = _render5.getByLabelText;
 
-    var passwordInput = getByRole("textbox");
+    var passwordInput = getByLabelText("Password input");
 
     _userEvent["default"].type(passwordInput, "Pa$$w0rd");
 
@@ -83,9 +83,9 @@ describe("Password input component tests", function () {
       clearable: true
     })),
         getAllByRole = _render6.getAllByRole,
-        getByRole = _render6.getByRole;
+        getByLabelText = _render6.getByLabelText;
 
-    var passwordInput = getByRole("textbox");
+    var passwordInput = getByLabelText("Password input");
 
     _userEvent["default"].type(passwordInput, "Pa$$w0rd");
 
@@ -101,9 +101,9 @@ describe("Password input component tests", function () {
       label: "Password input"
     })),
         getAllByRole = _render7.getAllByRole,
-        getByRole = _render7.getByRole;
+        getByLabelText = _render7.getByLabelText;
 
-    var passwordInput = getByRole("textbox");
+    var passwordInput = getByLabelText("Password input");
 
     _userEvent["default"].type(passwordInput, "Pa$$w0rd");
 
@@ -117,9 +117,9 @@ describe("Password input component tests", function () {
       clearable: true
     })),
         getAllByRole = _render8.getAllByRole,
-        getByRole = _render8.getByRole;
+        getByLabelText = _render8.getByLabelText;
 
-    var passwordInput = getByRole("textbox");
+    var passwordInput = getByLabelText("Password input");
 
     _userEvent["default"].type(passwordInput, "Pa$$w0rd");
 
@@ -164,9 +164,10 @@ describe("Password input component tests", function () {
     var _render10 = (0, _react2.render)( /*#__PURE__*/_react["default"].createElement(_PasswordInput["default"], {
       label: "Password input"
     })),
-        getByRole = _render10.getByRole;
+        getByRole = _render10.getByRole,
+        getByLabelText = _render10.getByLabelText;
 
-    var passwordInput = getByRole("textbox");
+    var passwordInput = getByLabelText("Password input");
     expect(passwordInput.getAttribute("aria-autocomplete")).toBeNull();
     expect(passwordInput.getAttribute("aria-controls")).toBeNull();
     expect(passwordInput.getAttribute("aria-expanded")).toBeNull();

package/quick-nav/QuickNav.js

@@ -48,13 +48,10 @@ var DxcQuickNav = function DxcQuickNav(_ref) {
   }, /*#__PURE__*/_react["default"].createElement(_Heading["default"], {
     level: 4,
     text: title || translatedLabels.quickNav.contentTitle
-  }), /*#__PURE__*/_react["default"].createElement(ListColumn, null, /*#__PURE__*/_react["default"].createElement(_Flex["default"], {
-    direction: "column",
-    gap: "0.5rem"
-  }, links.map(function (link) {
+  }), /*#__PURE__*/_react["default"].createElement(ListColumn, null, links.map(function (link) {
     var _link$links;
 
-    return /*#__PURE__*/_react["default"].createElement(ListRow, {
+    return /*#__PURE__*/_react["default"].createElement("li", {
       key: link.label
     }, /*#__PURE__*/_react["default"].createElement(_Inset["default"], {
       space: "0.25rem"
@@ -62,18 +59,20 @@ var DxcQuickNav = function DxcQuickNav(_ref) {
       href: "#".concat((0, _slugify["default"])(link === null || link === void 0 ? void 0 : link.label, {
         lower: true
       }))
-    }, link === null || link === void 0 ? void 0 : link.label), (_link$links = link.links) === null || _link$links === void 0 ? void 0 : _link$links.map(function (sublink) {
-      return /*#__PURE__*/_react["default"].createElement(ListRow, {
+    }, link === null || link === void 0 ? void 0 : link.label), /*#__PURE__*/_react["default"].createElement(ListSecondColumn, null, (_link$links = link.links) === null || _link$links === void 0 ? void 0 : _link$links.map(function (sublink) {
+      return /*#__PURE__*/_react["default"].createElement("li", {
         key: sublink.label
       }, /*#__PURE__*/_react["default"].createElement(_Inset["default"], {
         horizontal: "0.5rem"
       }, /*#__PURE__*/_react["default"].createElement(_Typography["default"], null, /*#__PURE__*/_react["default"].createElement(Link, {
-        href: "#".concat((0, _slugify["default"])(sublink === null || sublink === void 0 ? void 0 : sublink.label, {
+        href: "#".concat((0, _slugify["default"])(link === null || link === void 0 ? void 0 : link.label, {
+          lower: true
+        }), "-").concat((0, _slugify["default"])(sublink === null || sublink === void 0 ? void 0 : sublink.label, {
           lower: true
         }))
       }, sublink === null || sublink === void 0 ? void 0 : sublink.label))));
-    }))));
-  }))))));
+    })))));
+  })))));
 };
 
 var QuickNavContainer = _styledComponents["default"].div(_templateObject || (_templateObject = (0, _taggedTemplateLiteral2["default"])(["\n  padding-top: ", ";\n  padding-bottom: ", ";\n  padding-left: ", ";\n  padding-right: ", ";\n  border-left: 2px solid ", ";\n"])), function (props) {
@@ -88,9 +87,9 @@ var QuickNavContainer = _styledComponents["default"].div(_templateObject || (_te
   return props.theme.dividerBorderColor;
 });
 
-var ListColumn = _styledComponents["default"].ul(_templateObject2 || (_templateObject2 = (0, _taggedTemplateLiteral2["default"])(["\n  list-style-type: none;\n  margin: 0;\n  padding: 0;\n  width: 100%;\n"])));
+var ListColumn = _styledComponents["default"].ul(_templateObject2 || (_templateObject2 = (0, _taggedTemplateLiteral2["default"])(["\n  display: flex;\n  flex-direction: column;\n  gap: 0.5rem;\n  margin: 0;\n  padding: 0;\n  list-style-type: none;\n"])));
 
-var ListRow = _styledComponents["default"].li(_templateObject3 || (_templateObject3 = (0, _taggedTemplateLiteral2["default"])(["\n  width: 100%;\n"])));
+var ListSecondColumn = _styledComponents["default"].ul(_templateObject3 || (_templateObject3 = (0, _taggedTemplateLiteral2["default"])(["\n  display: flex;\n  flex-direction: column;\n  margin: 0;\n  padding: 0;\n  list-style-type: none;\n"])));
 
 var Link = _styledComponents["default"].a(_templateObject4 || (_templateObject4 = (0, _taggedTemplateLiteral2["default"])(["\n  text-decoration: none;\n  font-size: ", ";\n  font-family: ", ";\n  font-style: ", ";\n  font-weight: ", ";\n  color: ", ";\n  display: block;\n  text-overflow: ellipsis;\n  white-space: nowrap;\n  overflow: hidden;\n  width: fit-content;\n  max-width: 100%;\n\n  &:hover {\n    color: ", ";\n  }\n  &:focus {\n    outline-color: ", ";\n    outline-style: ", ";\n    outline-width: ", ";\n    border-radius: ", ";\n  }\n"])), function (props) {
   return props.theme.fontSize;