@zohodesk/dot 1.0.0-temp-187.25 → 1.0.0-temp-225

package/es/list/listCommon.module.css

@@ -41,7 +41,7 @@
 .classic .floatingIconContainer {
   display: none;
 }
-
+/*Active:ignore*/
 .compact .floatingIconContainer,
 .superCompact .floatingIconContainer,
 .listContainer:hover .classic .floatingIconContainer,
@@ -69,7 +69,7 @@
   position: relative;
   transition: var(--zd_transition3) all ease;
 }
-
+/*Active:ignore*/
 .listContainer:hover .compact .floatingIconContainer,
 .listContainer:hover .superCompact .floatingIconContainer,
 .hoveredStyle .compact .floatingIconContainer,

package/lib/AttachmentViewer/AttachmentViewer.js

@@ -543,7 +543,9 @@ var AttachmentViewer = /*#__PURE__*/function (_Component) {
           i18nKeys = _this$props2.i18nKeys,
           isActive = _this$props2.isActive,
           dataId = _this$props2.dataId,
-          customProps = _this$props2.customProps;
+          customProps = _this$props2.customProps,
+          showCount = _this$props2.showCount,
+          renderCustomIcons = _this$props2.renderCustomIcons;
       var _customProps$avatarPr = customProps.avatarProps,
           avatarProps = _customProps$avatarPr === void 0 ? {} : _customProps$avatarPr;
       var totalLen = data.length;
@@ -585,7 +587,7 @@ var AttachmentViewer = /*#__PURE__*/function (_Component) {
           "data-title": selectedAttachment.name,
           dataId: "attachName",
           id: ariaId
-        }, selectedAttachment.name), /*#__PURE__*/_react["default"].createElement(_Layout.Box, {
+        }, selectedAttachment.name), showCount && /*#__PURE__*/_react["default"].createElement(_Layout.Box, {
           flexible: true,
           className: _AttachmentViewerModule["default"].count,
           dataId: "attachCountContainer"
@@ -597,7 +599,11 @@ var AttachmentViewer = /*#__PURE__*/function (_Component) {
           align: uptoTablet ? 'bottom' : 'center',
           wrap: "wrap",
           isCover: false
-        }, canZoom ? isZoomed ? /*#__PURE__*/_react["default"].createElement("div", {
+        }, typeof renderCustomIcons === 'function' && renderCustomIcons({
+          selectedAttachment: selectedAttachment,
+          selectedIndex: selectedIndex,
+          totalLen: totalLen
+        }), canZoom ? isZoomed ? /*#__PURE__*/_react["default"].createElement("div", {
           className: uptoTablet ? _AttachmentViewerModule["default"].mobileMenu : _AttachmentViewerModule["default"].menu,
           onClick: _this6.zoomOut.bind(_this6, selectedIndex),
           "data-id": "zoomOut",

package/lib/AttachmentViewer/props/defaultProps.js

@@ -19,6 +19,7 @@ var AttachmentViewer_defaultProps = {
   isActive: false,
   dataId: 'AttachmentViewer',
   i18nKeys: {},
-  customProps: {}
+  customProps: {},
+  showCount: true
 };
 exports.AttachmentViewer_defaultProps = AttachmentViewer_defaultProps;

package/lib/AttachmentViewer/props/propTypes.js

@@ -56,6 +56,8 @@ var AttachmentViewer_propTypes = {
   renderUnSupportedElement: _propTypes["default"].func,
   customProps: _propTypes["default"].shape({
     avatarProps: _propTypes["default"].exact(_propTypes2.propTypes)
-  })
+  }),
+  showCount: _propTypes["default"].bool,
+  renderCustomIcons: _propTypes["default"].func
 };
 exports.AttachmentViewer_propTypes = AttachmentViewer_propTypes;

package/lib/list/DepartmentDropDown/DepartmentDropDown.js

@@ -105,9 +105,7 @@ var DepartmentDropDown = /*#__PURE__*/function (_Component) {
           i18nKeys = _this$props3.i18nKeys,
           customProps = _this$props3.customProps;
       var _customProps$ToggleDr = customProps.ToggleDropDownProps,
-          ToggleDropDownProps = _customProps$ToggleDr === void 0 ? {} : _customProps$ToggleDr,
-          _customProps$Departme = customProps.DepartmentTextProps,
-          DepartmentTextProps = _customProps$Departme === void 0 ? {} : _customProps$Departme;
+          ToggleDropDownProps = _customProps$ToggleDr === void 0 ? {} : _customProps$ToggleDr;
       var _i18nKeys$title = i18nKeys.title,
           title = _i18nKeys$title === void 0 ? 'Move Department' : _i18nKeys$title,
           _i18nKeys$searchEmpty = i18nKeys.searchEmptyText,
@@ -127,10 +125,10 @@ var DepartmentDropDown = /*#__PURE__*/function (_Component) {
       }, ToggleDropDownProps, {
         title: title,
         options: departmentList,
-        value: /*#__PURE__*/_react["default"].createElement(_SecondaryText.DepartmentText, _extends({
+        value: /*#__PURE__*/_react["default"].createElement(_SecondaryText.DepartmentText, {
           text: departmentName,
           dataTitle: departmentName
-        }, DepartmentTextProps)),
+        }),
         onClick: this.onMoveDepartment,
         isEditable: isEditable,
         onSelectLabel: onSelectLabel,

package/lib/list/DepartmentDropDown/props/propTypes.js

@@ -32,7 +32,6 @@ var propTypes = (_propTypes = {
   searchErrorText: _propTypes2["default"].string,
   placeholder: _propTypes2["default"].string
 })), _defineProperty(_propTypes, "customProps", _propTypes2["default"].shape({
-  ToggleDropDownProps: _propTypes2["default"].object,
-  DepartmentTextProps: _propTypes2["default"].object
+  ToggleDropDownProps: _propTypes2["default"].object
 })), _propTypes);
 exports.propTypes = propTypes;

package/lib/list/Subject/Subject.js

@@ -105,13 +105,12 @@ var Subject = /*#__PURE__*/function (_Component) {
         href: href,
         urlData: urlData,
         onClick: onClick,
-        target: target
+        target: target,
+        dataId: dataId
       }, LinkProps), children ? children : /*#__PURE__*/_react["default"].createElement(_Typography["default"], {
         $tagAttributes_text: linkTagAttributes,
         $i18n_dataTitle: text,
         $ui_className: "".concat(_SubjectModule["default"].subject, " ").concat(_cssUtils.whiteSpaceClassMapping[whiteSpace], " ").concat(className, " ").concat(_SubjectModule["default"].cursorPointer),
-        customId: dataId,
-        testId: dataId,
         $flag_dotted: isDotted,
         $ui_weight: fontWeight,
         $ui_highlightConfig: isHighlighted ? highlightData : _Common.DUMMY_OBJECT

package/lib/list/Subject/__tests__/Subject.spec.js

@@ -4,6 +4,8 @@ var _react = _interopRequireDefault(require("react"));
 
 var _react2 = require("@testing-library/react");
 
+require("@testing-library/jest-dom");
+
 var _Subject = _interopRequireDefault(require("../Subject"));
 
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { "default": obj }; }
@@ -24,4 +26,126 @@ describe('Subject', function () {
 
     expect(asFragment()).toMatchSnapshot();
   });
+  test.each(whiteSpace)('rendering the whiteSpace of -%s', function (whiteSpace) {
+    var _render3 = (0, _react2.render)( /*#__PURE__*/_react["default"].createElement(_Subject["default"], {
+      whiteSpace: whiteSpace,
+      isLink: true
+    })),
+        asFragment = _render3.asFragment;
+
+    expect(asFragment()).toMatchSnapshot();
+  });
+  test('renders data attributes on link wrapper', function () {
+    var dataId = 'subject-id';
+
+    var _render4 = (0, _react2.render)( /*#__PURE__*/_react["default"].createElement(_Subject["default"], {
+      isLink: true,
+      dataId: dataId
+    })),
+        container = _render4.container;
+
+    expect(container).toMatchSnapshot();
+    var anchor = container.querySelector('a');
+    expect(anchor.getAttribute('data-id')).toBe(dataId);
+    expect(anchor.getAttribute('data-test-id')).toBe(dataId);
+  });
+  test('renders text content', function () {
+    var text = 'Test Subject';
+
+    var _render5 = (0, _react2.render)( /*#__PURE__*/_react["default"].createElement(_Subject["default"], {
+      text: text
+    })),
+        container = _render5.container;
+
+    expect(container.textContent).toContain(text);
+  });
+  test('renders with custom className', function () {
+    var customClass = 'custom-subject';
+
+    var _render6 = (0, _react2.render)( /*#__PURE__*/_react["default"].createElement(_Subject["default"], {
+      className: customClass
+    })),
+        container = _render6.container;
+
+    expect(container.querySelector(".".concat(customClass))).not.toBeNull();
+  });
+  test('renders dotted text when isDotted is true', function () {
+    var _render7 = (0, _react2.render)( /*#__PURE__*/_react["default"].createElement(_Subject["default"], {
+      text: "Test",
+      isDotted: true
+    })),
+        asFragment = _render7.asFragment;
+
+    expect(asFragment()).toMatchSnapshot();
+  });
+  test('renders children when provided', function () {
+    var _render8 = (0, _react2.render)( /*#__PURE__*/_react["default"].createElement(_Subject["default"], {
+      isLink: true
+    }, /*#__PURE__*/_react["default"].createElement("span", null, "Custom Child"))),
+        container = _render8.container;
+
+    expect(container.querySelector('span').textContent).toBe('Custom Child');
+  });
+  test('applies fontWeight prop', function () {
+    var _render9 = (0, _react2.render)( /*#__PURE__*/_react["default"].createElement(_Subject["default"], {
+      text: "Test",
+      fontWeight: "bold"
+    })),
+        asFragment = _render9.asFragment;
+
+    expect(asFragment()).toMatchSnapshot();
+  });
+  test('renders with href when isLink is true', function () {
+    var href = 'https://example.com';
+
+    var _render10 = (0, _react2.render)( /*#__PURE__*/_react["default"].createElement(_Subject["default"], {
+      isLink: true,
+      href: href
+    })),
+        container = _render10.container;
+
+    expect(container.querySelector('a').getAttribute('href')).toBe(href);
+  });
+  test('calls onClick when link is clicked', function () {
+    var handleClick = jest.fn();
+
+    var _render11 = (0, _react2.render)( /*#__PURE__*/_react["default"].createElement(_Subject["default"], {
+      isLink: true,
+      onClick: handleClick
+    })),
+        container = _render11.container;
+
+    container.querySelector('a').click();
+    expect(handleClick).toHaveBeenCalled();
+  });
+  test('renders as non-link when isLink is false', function () {
+    var _render12 = (0, _react2.render)( /*#__PURE__*/_react["default"].createElement(_Subject["default"], {
+      text: "Test",
+      isLink: false
+    })),
+        container = _render12.container;
+
+    expect(container.querySelector('a')).toBeNull();
+  });
+  test('applies custom className to non-link element', function () {
+    var customClass = 'test-class';
+
+    var _render13 = (0, _react2.render)( /*#__PURE__*/_react["default"].createElement(_Subject["default"], {
+      text: "Test",
+      className: customClass,
+      isLink: false
+    })),
+        container = _render13.container;
+
+    expect(container.querySelector(".".concat(customClass))).not.toBeNull();
+  });
+  test('renders with target prop on link', function () {
+    var _render14 = (0, _react2.render)( /*#__PURE__*/_react["default"].createElement(_Subject["default"], {
+      isLink: true,
+      target: "_blank"
+    })),
+        container = _render14.container;
+
+    expect(container.querySelector('a').getAttribute('target')).toBe('_blank');
+  });
 });

package/lib/list/Subject/__tests__/__snapshots__/Subject.spec.js.snap

@@ -1,5 +1,17 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
+exports[`Subject applies fontWeight prop 1`] = `
+<DocumentFragment>
+  <div
+    class="dotted font_bold subject  whiteSpace_pre"
+    data-title="Test"
+    data-title-wrap="pre"
+  >
+    Test
+  </div>
+</DocumentFragment>
+`;
+
 exports[`Subject rendering the defult props 1`] = `
 <DocumentFragment>
   <div
@@ -18,6 +30,21 @@ exports[`Subject rendering the whiteSpace of -break-spaces 1`] = `
 </DocumentFragment>
 `;
 
+exports[`Subject rendering the whiteSpace of -break-spaces 2`] = `
+<DocumentFragment>
+  <a
+    class="container  "
+    href="javascript:void(0);"
+    rel="noopener noreferrer"
+  >
+    <div
+      class="dotted font_regular subject whiteSpace_breakSpaces  cursorPointer"
+      data-title-wrap="break-spaces"
+    />
+  </a>
+</DocumentFragment>
+`;
+
 exports[`Subject rendering the whiteSpace of -normal 1`] = `
 <DocumentFragment>
   <div
@@ -27,6 +54,21 @@ exports[`Subject rendering the whiteSpace of -normal 1`] = `
 </DocumentFragment>
 `;
 
+exports[`Subject rendering the whiteSpace of -normal 2`] = `
+<DocumentFragment>
+  <a
+    class="container  "
+    href="javascript:void(0);"
+    rel="noopener noreferrer"
+  >
+    <div
+      class="dotted font_regular subject whiteSpace_normal  cursorPointer"
+      data-title-wrap="normal"
+    />
+  </a>
+</DocumentFragment>
+`;
+
 exports[`Subject rendering the whiteSpace of -nowrap 1`] = `
 <DocumentFragment>
   <div
@@ -36,6 +78,21 @@ exports[`Subject rendering the whiteSpace of -nowrap 1`] = `
 </DocumentFragment>
 `;
 
+exports[`Subject rendering the whiteSpace of -nowrap 2`] = `
+<DocumentFragment>
+  <a
+    class="container  "
+    href="javascript:void(0);"
+    rel="noopener noreferrer"
+  >
+    <div
+      class="dotted font_regular subject whiteSpace_nowrap  cursorPointer"
+      data-title-wrap="nowrap"
+    />
+  </a>
+</DocumentFragment>
+`;
+
 exports[`Subject rendering the whiteSpace of -pre 1`] = `
 <DocumentFragment>
   <div
@@ -45,6 +102,21 @@ exports[`Subject rendering the whiteSpace of -pre 1`] = `
 </DocumentFragment>
 `;
 
+exports[`Subject rendering the whiteSpace of -pre 2`] = `
+<DocumentFragment>
+  <a
+    class="container  "
+    href="javascript:void(0);"
+    rel="noopener noreferrer"
+  >
+    <div
+      class="dotted font_regular subject whiteSpace_pre  cursorPointer"
+      data-title-wrap="pre"
+    />
+  </a>
+</DocumentFragment>
+`;
+
 exports[`Subject rendering the whiteSpace of -pre-line 1`] = `
 <DocumentFragment>
   <div
@@ -54,6 +126,21 @@ exports[`Subject rendering the whiteSpace of -pre-line 1`] = `
 </DocumentFragment>
 `;
 
+exports[`Subject rendering the whiteSpace of -pre-line 2`] = `
+<DocumentFragment>
+  <a
+    class="container  "
+    href="javascript:void(0);"
+    rel="noopener noreferrer"
+  >
+    <div
+      class="dotted font_regular subject whiteSpace_preLine  cursorPointer"
+      data-title-wrap="pre-line"
+    />
+  </a>
+</DocumentFragment>
+`;
+
 exports[`Subject rendering the whiteSpace of -pre-wrap 1`] = `
 <DocumentFragment>
   <div
@@ -62,3 +149,47 @@ exports[`Subject rendering the whiteSpace of -pre-wrap 1`] = `
   />
 </DocumentFragment>
 `;
+
+exports[`Subject rendering the whiteSpace of -pre-wrap 2`] = `
+<DocumentFragment>
+  <a
+    class="container  "
+    href="javascript:void(0);"
+    rel="noopener noreferrer"
+  >
+    <div
+      class="dotted font_regular subject whiteSpace_preWrap  cursorPointer"
+      data-title-wrap="pre-wrap"
+    />
+  </a>
+</DocumentFragment>
+`;
+
+exports[`Subject renders data attributes on link wrapper 1`] = `
+<div>
+  <a
+    class="container  "
+    data-id="subject-id"
+    data-test-id="subject-id"
+    href="javascript:void(0);"
+    rel="noopener noreferrer"
+  >
+    <div
+      class="dotted font_regular subject whiteSpace_pre  cursorPointer"
+      data-title-wrap="pre"
+    />
+  </a>
+</div>
+`;
+
+exports[`Subject renders dotted text when isDotted is true 1`] = `
+<DocumentFragment>
+  <div
+    class="dotted font_regular subject  whiteSpace_pre"
+    data-title="Test"
+    data-title-wrap="pre"
+  >
+    Test
+  </div>
+</DocumentFragment>
+`;

package/lib/list/Subject/props/defaultProps.js

@@ -4,14 +4,17 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.defaultProps = void 0;
+
+var _constants = require("@zohodesk/dotkit/es/utils/constants");
+
 var defaultProps = {
   isLink: false,
   fontWeight: 'regular',
   className: '',
   isDotted: true,
-  customProps: {},
+  customProps: _constants.DUMMY_OBJECT,
   whiteSpace: 'pre',
   isHighlighted: false,
-  highlightData: {}
+  highlightData: _constants.DUMMY_OBJECT
 };
 exports.defaultProps = defaultProps;

package/lib/list/listCommon.module.css

@@ -41,7 +41,7 @@
 .classic .floatingIconContainer {
   display: none;
 }
-
+/*Active:ignore*/
 .compact .floatingIconContainer,
 .superCompact .floatingIconContainer,
 .listContainer:hover .classic .floatingIconContainer,
@@ -69,7 +69,7 @@
   position: relative;
   transition: var(--zd_transition3) all ease;
 }
-
+/*Active:ignore*/
 .listContainer:hover .compact .floatingIconContainer,
 .listContainer:hover .superCompact .floatingIconContainer,
 .hoveredStyle .compact .floatingIconContainer,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zohodesk/dot",
-  "version": "1.0.0-temp-187.25",
+  "version": "1.0.0-temp-225",
   "main": "lib/index",
   "module": "es/index.js",
   "private": false,
@@ -34,7 +34,7 @@
     "snap-update": "npm run test-clean && npm run test -- -u",
     "sstest": "react-cli sstest",
     "common_package_build": "cd ../common && npm run build && cd ../dot",
-    "prepublishOnly": "node prePublish.js && npm run css:review  && npm run review:props",
+    "prepublishOnly": "node prePublish.js && npm run downloadOnly && npm run css:review  && npm run review:props",
     "postpublish": "node postPublish.js",
     "test-clean": "react-cli clean ./coverage && react-cli clean ./unittest react-cli clean ./es && react-cli clean ./lib &&  react-cli clean ./package-lock.json &&  react-cli clean ./result.json",
     "download": "npm run downloadOnly  && cd ../ && npm run download",
@@ -68,9 +68,9 @@
     "@zohodesk-private/node-plugins": "1.1.13",
     "@zohodesk-private/react-prop-validator": "1.2.3",
     "@zohodesk/a11y": "2.3.8",
-    "@zohodesk/components": "1.6.3",
-    "@zohodesk/hooks": "2.0.6",
-    "@zohodesk/icons": "1.2.4",
+    "@zohodesk/components": "1.6.4",
+    "@zohodesk/hooks": "2.0.8",
+    "@zohodesk/icons": "1.2.5",
     "@zohodesk/layout": "^3.1.0",
     "@zohodesk/svg": "1.3.2",
     "@zohodesk/utils": "1.3.16",
@@ -78,19 +78,19 @@
     "@zohodesk/virtualizer": "1.0.13",
     "react-sortable-hoc": "^0.8.3",
     "velocity-react": "1.4.3",
-    "@zohodesk/dotkit": "1.0.7",
+    "@zohodesk/dotkit": "1.0.8",
     "@zohodesk/react-cli": "1.1.27",
     "@zohodesk/client_build_tool": "0.0.20"
   },
   "peerDependencies": {
     "velocity-react": "1.4.3",
     "@zohodesk/variables": "1.2.0",
-    "@zohodesk/components": "1.6.3",
-    "@zohodesk/icons": "1.2.4",
+    "@zohodesk/components": "1.6.4",
+    "@zohodesk/icons": "1.2.5",
     "@zohodesk/svg": "1.3.2",
     "@zohodesk/virtualizer": "1.0.13",
     "react-sortable-hoc": "^0.8.3",
-    "@zohodesk/hooks": "2.0.6",
+    "@zohodesk/hooks": "2.0.8",
     "@zohodesk/utils": "1.3.16",
     "@zohodesk/a11y": "2.3.8",
     "@zohodesk/layout": "3.1.0"