@zohodesk/components 1.0.0-temp-180 → 1.0.0-temp-181

package/es/v1/ListItem/ListContainer.js

@@ -89,7 +89,7 @@ const ListContainer = props => {
     eleRef: eleRef,
     tagName: isLink ? 'a' : 'li',
     "data-title": isDisabled ? disableTitle : title
-  }, options, customProps, a11y), children);
+  }, options, customProps), children);
 };
 
 ListContainer.defaultProps = ListContainerDefaultProps;

package/es/v1/MultiSelect/AdvancedGroupMultiSelect.js

@@ -409,8 +409,8 @@ class AdvancedGroupMultiSelect extends React.Component {
     if (suggestionsLen && isPopupOpen && keyCode === 38) {
       //up arrow
 
-      /*if (hoverOption === 0) { //disable first to last option higlight
-        !isNextOptions && this.setState({ hoverOption: suggestionsLen - 1 });
+      /*if (hoverOption === 0) { //disable first to last option higlight
+        !isNextOptions && this.setState({ hoverOption: suggestionsLen - 1 });
       }*/
       if (hoverIndex) {
         this.setState({
@@ -420,9 +420,9 @@ class AdvancedGroupMultiSelect extends React.Component {
     } else if (suggestionsLen && isPopupOpen && keyCode === 40) {
       //down arrow
 
-      /*else if (hoverOption === suggestionsLen - 1 || hoverOption === null) {
-        //disable last to first option higlight
-        !isNextOptions && this.setState({ hoverOption: 0 });
+      /*else if (hoverOption === suggestionsLen - 1 || hoverOption === null) {
+        //disable last to first option higlight
+        !isNextOptions && this.setState({ hoverOption: 0 });
       }*/
       if (isNextOptions && suggestionsLen >= 5 && hoverIndex === suggestionsLen - 3) {
         getNextOptions && this.handleFetchOptions(getNextOptions, searchStr);

package/es/v1/MultiSelect/MultiSelect.js

@@ -321,8 +321,8 @@ export class MultiSelectComponent extends React.Component {
     if (suggestionsLen && isPopupOpen && keyCode === 38) {
       //up arrow
 
-      /*if (hoverOption === 0) { //disable first to last option higlight
-        !isNextOptions && this.setState({ hoverOption: suggestionsLen - 1 });
+      /*if (hoverOption === 0) { //disable first to last option higlight
+        !isNextOptions && this.setState({ hoverOption: suggestionsLen - 1 });
       }*/
       if (hoverOption) {
         this.setState({
@@ -332,9 +332,9 @@ export class MultiSelectComponent extends React.Component {
     } else if (suggestionsLen && isPopupOpen && keyCode === 40) {
       //down arrow
 
-      /*else if (hoverOption === suggestionsLen - 1 || hoverOption === null) {
-        //disable last to first option higlight
-        !isNextOptions && this.setState({ hoverOption: 0 });
+      /*else if (hoverOption === suggestionsLen - 1 || hoverOption === null) {
+        //disable last to first option higlight
+        !isNextOptions && this.setState({ hoverOption: 0 });
       }*/
       if (isNextOptions && suggestionsLen >= 5 && hoverOption === suggestionsLen - 3) {
         getNextOptions && this.handleFetchOptions(getNextOptions, searchStr);

package/es/v1/MultiSelect/Suggestions.js

@@ -60,8 +60,7 @@ export default class Suggestions extends React.PureComponent {
       const isHighlight = hoverOption === index || id === hoverId ? true : false;
       const list_a11y = Object.assign({}, a11y, {
         ariaSelected: isActive,
-        ariaLabel: value,
-        'data-a11y-list-active': isHighlight
+        ariaLabel: value
       });
       const commonProps = {
         isDisabled,

package/es/v1/Select/Select.js

@@ -298,8 +298,7 @@ export class SelectComponent extends Component {
       isPopupOpen,
       onKeyDown,
       isPopupOpenOnEnter,
-      needCloseOnSelect,
-      isPopupReady
+      needCloseOnSelect
     } = this.props;
     let {
       hoverIndex,
@@ -352,7 +351,7 @@ export class SelectComponent extends Component {
         id
       } = option || {};
 
-      if (isPopupReady && !getIsEmptyValue(id) && onChange) {
+      if (isPopupOpen && !getIsEmptyValue(id) && onChange) {
         onChange(id, optionsNormalize[id]);
         needCloseOnSelect && this.handlePopupClose(e);
       }

package/es/v1/Switch/Switch.js

@@ -54,6 +54,7 @@ const Switch = props => {
     "data-title": disabled ? disableTitle : title,
     "aria-checked": checked,
     role: "switch",
+    tabIndex: isReadOnly || disabled ? '-1' : '0',
     dataSelectorId: dataSelectorId
   }, SwitchProps), /*#__PURE__*/React.createElement(Box, {
     className: `${style[size]} ${customSwitchSize}`
@@ -71,8 +72,7 @@ const Switch = props => {
     htmlFor: id,
     "data-id": dataId,
     "data-test-id": dataId,
-    className: `${style.label} ${style[`${size}Label`]} ${customSwitch}`,
-    tabIndex: isReadOnly || disabled ? '-1' : '0'
+    className: `${style.label} ${style[`${size}Label`]} ${customSwitch}`
   })), text && /*#__PURE__*/React.createElement(Label, _extends({
     text: text,
     palette: labelPalette,

package/es/v1/Tab/Tabs.js

@@ -2,12 +2,12 @@ function _extends() { _extends = Object.assign ? Object.assign.bind() : function
 
 /* eslint-disable react/forbid-component-props */
 
-/* eslint css-modules/no-unused-class: [2, {
-      markAsUsed: [
-          'hidden','alpha','gamma','beta','delta','special','text','maxWidth','widgetList','menuContainer','border','tabText','block','tabAlpha','alphaActive','gammaActive','betaActive','deltaActive','tabGamma','tabBeta','tabDelta','specialActive','tabSpecial',
-          'alpha_padding','alpha_border','gamma_padding','gamma_border','beta_padding','beta_border','delta_padding','delta_border','textContainer','disabled','alphaActive_border','gammaActive_border','betaActive_border','deltaActive_border'
-        ]
-    }]
+/* eslint css-modules/no-unused-class: [2, {
+      markAsUsed: [
+          'hidden','alpha','gamma','beta','delta','special','text','maxWidth','widgetList','menuContainer','border','tabText','block','tabAlpha','alphaActive','gammaActive','betaActive','deltaActive','tabGamma','tabBeta','tabDelta','specialActive','tabSpecial',
+          'alpha_padding','alpha_border','gamma_padding','gamma_border','beta_padding','beta_border','delta_padding','delta_border','textContainer','disabled','alphaActive_border','gammaActive_border','betaActive_border','deltaActive_border'
+        ]
+    }]
 */
 import React from 'react';
 import Tab from './Tab';
@@ -141,9 +141,9 @@ class Tabs extends React.Component {
     } = this.state;
 
     if (prevProps.children && children) {
-      /**
-       * To recalculate the dimensions of tabs we are checking the children length.
-       * and also checking the tab id and their order.
+      /**
+       * To recalculate the dimensions of tabs we are checking the children length.
+       * and also checking the tab id and their order.
        */
       let newTabKeys = [];
       React.Children.toArray(children).forEach(child => {
@@ -316,14 +316,14 @@ class Tabs extends React.Component {
       });
 
       if (selectedTabDimension > totalDimension) {
-        /* let { align } = this.props;
-        let newDim = `${remConvert(totalDimension)}rem`;
-        let newTabs;
-        if (align === 'vertical') {
-          newTabs = mainTabs.map(tab => React.cloneElement(tab, { style: { maxWidth: newDim } }));
-        } else {
-          newTabs = mainTabs.map(tab => React.cloneElement(tab, { style: { maxHeight: newDim } }));
-        }
+        /* let { align } = this.props;
+        let newDim = `${remConvert(totalDimension)}rem`;
+        let newTabs;
+        if (align === 'vertical') {
+          newTabs = mainTabs.map(tab => React.cloneElement(tab, { style: { maxWidth: newDim } }));
+        } else {
+          newTabs = mainTabs.map(tab => React.cloneElement(tab, { style: { maxHeight: newDim } }));
+        }
         mainTabs = newTabs; */
         this.setMaxDim(totalDimension);
       }

package/es/v1/Tag/Tag.js

@@ -7,16 +7,16 @@ import { Container } from '../Layout';
 import Button from '../semantic/Button/Button';
 import { useUniqueId } from '../../Provider/IdProvider';
 import style from '../../Tag/Tag.module.css';
-/*
- * eslint css-modules/no-unused-class: [2, { markAsUsed: ['small', 'medium',
- * 'closedanger', 'closeprimary', 'default', 'danger', 'secondary',
- * 'pureDotted', 'primaryDotted'] }]
+/*
+ * eslint css-modules/no-unused-class: [2, { markAsUsed: ['small', 'medium',
+ * 'closedanger', 'closeprimary', 'default', 'danger', 'secondary',
+ * 'pureDotted', 'primaryDotted'] }]
  */
 
-/*
- * eslint css-modules/no-unused-class: [2, { markAsUsed: ['small', 'medium',
- * 'closedanger', 'closeprimary', 'default', 'danger', 'secondary',
- * 'pureDotted', 'primaryDotted'] }]
+/*
+ * eslint css-modules/no-unused-class: [2, { markAsUsed: ['small', 'medium',
+ * 'closedanger', 'closeprimary', 'default', 'danger', 'secondary',
+ * 'pureDotted', 'primaryDotted'] }]
  */
 
 function Tag(props) {