@elastic/eui 115.0.0 → 116.0.0-snapshot.1778594703965

package/es/components/collapsible_nav_beta/_kibana_solution/collapsible_nav_kibana_solution.js

@@ -66,7 +66,6 @@ export var KibanaCollapsibleNavSolution = function KibanaCollapsibleNavSolution(
   var solutionSolutionSwitcherAriaLabel = useEuiI18n('euiCollapsibleNavKibanaSolution.switcherAriaLabel', ' - click to switch to another solution');
   var solutionSolutionGroupLabel = useEuiI18n('euiCollapsibleNavKibanaSolution.groupLabel', 'Navigate to solution');
   var closeSolutionPopover = useCallback(function (event) {
-    var _childItem$firstEleme;
     // Allow child items to stop the popover from being closed
     if (event.isPropagationStopped()) return;
     // Only listen for clicks on child items - currentTarget is the parent <ul>
@@ -75,7 +74,7 @@ export var KibanaCollapsibleNavSolution = function KibanaCollapsibleNavSolution(
     // Only close the popover if the item is a clickable link or button
     var target = event.target;
     var childItem = target.closest('.euiListGroupItem');
-    if (['A', 'BUTTON'].includes((childItem === null || childItem === void 0 || (_childItem$firstEleme = childItem.firstElementChild) === null || _childItem$firstEleme === void 0 ? void 0 : _childItem$firstEleme.tagName) || '')) {
+    if (['A', 'BUTTON'].includes((childItem === null || childItem === void 0 ? void 0 : childItem.tagName) || '')) {
       setIsSolutionSwitcherOpen(false);
     }
   }, []);
@@ -88,12 +87,10 @@ export var KibanaCollapsibleNavSolution = function KibanaCollapsibleNavSolution(
       "data-test-subj": "kibanaSolutionSwitcherList",
       "aria-label": solutionSolutionGroupLabel,
       listItems: primaryItems,
-      size: "s",
       bordered: true,
       onClick: closeSolutionPopover
     }), secondaryItems.length > 0 && ___EmotionJSX(React.Fragment, null, ___EmotionJSX(EuiListGroup, {
       listItems: secondaryItems,
-      size: "s",
       css: styles.euiCollapsibleNavKibanaSolution__secondaryItems,
       onClick: closeSolutionPopover
     })));
@@ -174,22 +171,28 @@ KibanaCollapsibleNavSolution.propTypes = {
     "data-test-subj": PropTypes.string,
     css: PropTypes.any,
     /**
-         * Size of the label text
-         */
-    size: PropTypes.any,
+       * Controls the disabled behavior via the native `disabled` attribute.
+       */
+    isDisabled: PropTypes.bool,
+    /**
+       * NOTE: Beta feature, may be changed or removed in the future
+       *
+       * Changes the native `disabled` attribute to `aria-disabled` to preserve focusability.
+       * This results in a semantically disabled button without the default browser handling of the disabled state.
+       *
+       * Use e.g. when a disabled button should have a tooltip.
+       */
+    hasAriaDisabled: PropTypes.bool,
     /**
          * By default the item will get the color `text`.
          * You can customize the color of the item by passing a color name.
+         * @default 'text'
          */
     color: PropTypes.any,
     /**
          * Apply styles indicating an item is active
          */
     isActive: PropTypes.bool,
-    /**
-         * Apply styles indicating an item is disabled
-         */
-    isDisabled: PropTypes.bool,
     /**
          * Make the list item label a link.
          * While permitted, `href` and `onClick` should not be used together in most cases and may create problems.

package/es/components/color_picker/color_palette_picker/color_palette_picker.js

@@ -92,7 +92,6 @@ export var EuiColorPalettePicker = function EuiColorPalettePicker(_ref) {
     options: paletteOptions,
     valueOfSelected: valueOfSelected,
     onChange: onChange,
-    hasDividers: true,
     isInvalid: isInvalid,
     compressed: compressed,
     disabled: disabled,

package/es/components/combo_box/combo_box.a11y.js

@@ -87,7 +87,7 @@ describe('EuiComboBox', function () {
           rowHeight: rowHeight
         }));
         cy.get('input[data-test-subj="comboBoxSearchInput"]').realClick();
-        cy.get('button[data-test-subj="titanOption"]').should('exist');
+        cy.get('[data-test-subj="titanOption"]').should('exist');
         cy.checkAxe();
       });
       it('has zero violations after keyboard interaction', function () {
@@ -96,7 +96,7 @@ describe('EuiComboBox', function () {
         }));
         cy.realPress('Tab');
         cy.get('input[data-test-subj="comboBoxSearchInput"]').should('have.focus');
-        cy.get('button[data-test-subj="titanOption"]').should('exist');
+        cy.get('[data-test-subj="titanOption"]').should('exist');
         cy.repeatRealPress('ArrowDown');
         cy.realPress('Enter');
         cy.repeatRealPress('ArrowDown');
@@ -122,14 +122,14 @@ describe('EuiComboBox', function () {
         }));
         cy.realPress('Tab');
         cy.get('input[data-test-subj="comboBoxSearchInput"]').should('have.focus');
-        cy.get('button[data-test-subj="titanOption"]').should('exist');
+        cy.get('[data-test-subj="titanOption"]').should('exist');
         cy.realPress('ArrowDown');
         cy.realPress('ArrowDown');
         cy.realPress('ArrowDown');
         cy.get('input[data-test-subj="comboBoxSearchInput"]').invoke('attr', 'aria-activedescendant').should('include', 'option-2');
         cy.realPress('Enter');
         cy.realPress('ArrowDown');
-        cy.get('input[data-test-subj="comboBoxSearchInput"]').invoke('attr', 'aria-activedescendant').should('include', 'option-3');
+        cy.get('input[data-test-subj="comboBoxSearchInput"]').invoke('attr', 'aria-activedescendant').should('include', 'option-4');
       });
       it('sets the correct aria-activedescendant id with custom option ids', function () {
         cy.realMount(___EmotionJSX(ComboBox, {
@@ -159,7 +159,7 @@ describe('EuiComboBox', function () {
         cy.get('input[data-test-subj="comboBoxSearchInput"]').should('exist');
         cy.realPress('Tab');
         cy.get('input[data-test-subj="comboBoxSearchInput"]').should('have.focus');
-        cy.get('button[data-test-subj="titanOption"]').should('exist');
+        cy.get('[data-test-subj="titanOption"]').should('exist');
         cy.realPress('ArrowDown');
         cy.realPress('ArrowDown');
         cy.realPress('ArrowDown');

package/es/components/combo_box/combo_box.js

@@ -1,4 +1,4 @@
-var _excluded = ["data-test-subj", "async", "className", "compressed", "customOptionText", "fullWidth", "id", "inputRef", "isCaseSensitive", "isClearable", "isDisabled", "isInvalid", "isLoading", "noSuggestions", "onBlur", "onChange", "onCreateOption", "onSearchChange", "options", "placeholder", "renderOption", "rowHeight", "selectedOptions", "singleSelection", "prepend", "sortMatchesBy", "delimiter", "append", "autoFocus", "truncationProps", "inputPopoverProps", "optionMatcher", "aria-label", "aria-labelledby", "aria-describedby"];
+var _excluded = ["data-test-subj", "async", "className", "compressed", "customOptionText", "fullWidth", "id", "inputRef", "isCaseSensitive", "isClearable", "isDisabled", "isInvalid", "isLoading", "noSuggestions", "onBlur", "onChange", "onCreateOption", "onSearchChange", "options", "placeholder", "renderOption", "rowHeight", "selectedOptions", "singleSelection", "prepend", "sortMatchesBy", "delimiter", "append", "autoFocus", "onFocusBadge", "truncationProps", "inputPopoverProps", "optionMatcher", "aria-label", "aria-labelledby", "aria-describedby"];
 function _typeof(o) { "@babel/helpers - typeof"; return _typeof = "function" == typeof Symbol && "symbol" == typeof Symbol.iterator ? function (o) { return typeof o; } : function (o) { return o && "function" == typeof Symbol && o.constructor === Symbol && o !== Symbol.prototype ? "symbol" : typeof o; }, _typeof(o); }
 function _extends() { return _extends = Object.assign ? Object.assign.bind() : function (n) { for (var e = 1; e < arguments.length; e++) { var t = arguments[e]; for (var r in t) ({}).hasOwnProperty.call(t, r) && (n[r] = t[r]); } return n; }, _extends.apply(null, arguments); }
 function _objectWithoutProperties(e, t) { if (null == e) return {}; var o, r, i = _objectWithoutPropertiesLoose(e, t); if (Object.getOwnPropertySymbols) { var n = Object.getOwnPropertySymbols(e); for (r = 0; r < n.length; r++) o = n[r], t.indexOf(o) >= 0 || {}.propertyIsEnumerable.call(e, o) && (i[o] = e[o]); } return i; }
@@ -101,6 +101,9 @@ export var EuiComboBox = /*#__PURE__*/function (_Component) {
       _this.listRefInstance = ref;
     });
     _defineProperty(_this, "setListOptionRefs", function (node, index) {
+      var current = _this.state.listOptionRefs[index];
+      // Skip updating if the ref is null (on cleanup) or didn't change
+      if (node === null || node === current) return;
       _this.setState(function (_ref) {
         var listOptionRefs = _ref.listOptionRefs;
         var _listOptionRefs = listOptionRefs;
@@ -121,41 +124,32 @@ export var EuiComboBox = /*#__PURE__*/function (_Component) {
         isListOpen: false
       });
     });
-    _defineProperty(_this, "incrementActiveOptionIndex", function (amount) {
-      // If there are no options available, do nothing.
-      if (!_this.state.matchingOptions.length) {
-        return;
+    _defineProperty(_this, "findNextSelectableOptionIndex", function (options, startIndex) {
+      var direction = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : 1;
+      if (!options.length) return -1;
+      var index = startIndex;
+      for (var count = 0; count < options.length; count++) {
+        var option = options[index];
+        if (!option.isGroupLabelOption && !option.disabled) {
+          return index;
+        }
+        index = (index + direction + options.length) % options.length;
       }
+      return -1; // the remaining options can't be selected (group labels or disabled)
+    });
+    _defineProperty(_this, "incrementActiveOptionIndex", function (amount) {
+      if (!_this.state.matchingOptions.length) return;
       _this.setState(function (_ref2) {
         var activeOptionIndex = _ref2.activeOptionIndex,
           matchingOptions = _ref2.matchingOptions;
-        var nextActiveOptionIndex;
-        if (activeOptionIndex < 0) {
-          // If this is the beginning of the user's keyboard navigation of the menu, then we'll focus
-          // either the first or last item.
-          nextActiveOptionIndex = amount < 0 ? matchingOptions.length - 1 : 0;
-        } else {
-          nextActiveOptionIndex = activeOptionIndex + amount;
-          if (nextActiveOptionIndex < 0) {
-            nextActiveOptionIndex = matchingOptions.length - 1;
-          } else if (nextActiveOptionIndex === matchingOptions.length) {
-            nextActiveOptionIndex = 0;
-          }
-        }
-
-        // Group titles are included in option list but are not selectable
-        // Skip group title options
         var direction = amount > 0 ? 1 : -1;
-        while (matchingOptions[nextActiveOptionIndex].isGroupLabelOption) {
-          nextActiveOptionIndex = nextActiveOptionIndex + direction;
-          if (nextActiveOptionIndex < 0) {
-            nextActiveOptionIndex = matchingOptions.length - 1;
-          } else if (nextActiveOptionIndex === matchingOptions.length) {
-            nextActiveOptionIndex = 0;
-          }
-        }
+        var startIndex = activeOptionIndex < 0 ?
+        // Inintial interaction: jump to first or last item
+        amount < 0 ? matchingOptions.length - 1 : 0 :
+        // Advance by amount, wrapping around
+        (activeOptionIndex + amount + matchingOptions.length) % matchingOptions.length;
         return {
-          activeOptionIndex: nextActiveOptionIndex
+          activeOptionIndex: _this.findNextSelectableOptionIndex(matchingOptions, startIndex, direction)
         };
       });
     });
@@ -373,27 +367,36 @@ export var EuiComboBox = /*#__PURE__*/function (_Component) {
         onChange = _this$props7.onChange,
         selectedOptions = _this$props7.selectedOptions,
         singleSelectionProp = _this$props7.singleSelection;
+      var _this$state2 = _this.state,
+        matchingOptions = _this$state2.matchingOptions,
+        listOptionRefs = _this$state2.listOptionRefs;
       var singleSelection = Boolean(singleSelectionProp);
       var changeOptions = singleSelection ? [addedOption] : selectedOptions.concat(addedOption);
       onChange === null || onChange === void 0 || onChange(changeOptions);
       _this.clearSearchValue();
-      _this.clearActiveOption();
-      if (!isContainerBlur) {
-        var _this$searchInputRefI2;
-        (_this$searchInputRefI2 = _this.searchInputRefInstance) === null || _this$searchInputRefI2 === void 0 || _this$searchInputRefI2.focus();
-      }
       if (singleSelection) {
+        // List closes after single selection; return focus to the input
+        _this.clearActiveOption();
+        if (!isContainerBlur) {
+          var _this$searchInputRefI2;
+          (_this$searchInputRefI2 = _this.searchInputRefInstance) === null || _this$searchInputRefI2 === void 0 || _this$searchInputRefI2.focus();
+        }
         requestAnimationFrame(function () {
           return _this.closeList();
         });
+      } else if (isContainerBlur) {
+        // User tabbed away. `onContainerBlur` will close the list. We're only cleaning up state
+        _this.clearActiveOption();
       } else {
-        _this.setState(function (_ref4) {
-          var listOptionRefs = _ref4.listOptionRefs,
-            matchingOptions = _ref4.matchingOptions;
-          return {
-            listOptionRefs: listOptionRefs.slice(0, matchingOptions.length - 1),
-            activeOptionIndex: matchingOptions.indexOf(addedOption)
-          };
+        var currentIndex = matchingOptions.indexOf(addedOption);
+        var nextOptions = matchingOptions.filter(function (option) {
+          return option !== addedOption;
+        });
+        _this.setState({
+          listOptionRefs: listOptionRefs.slice(0, matchingOptions.length - 1),
+          activeOptionIndex: _this.findNextSelectableOptionIndex(nextOptions, Math.min(currentIndex, nextOptions.length - 1)
+          // direction defaults to 1 (forward)
+          )
         });
       }
     });
@@ -501,6 +504,7 @@ export var EuiComboBox = /*#__PURE__*/function (_Component) {
         delimiter = _this$props11.delimiter,
         append = _this$props11.append,
         autoFocus = _this$props11.autoFocus,
+        onFocusBadge = _this$props11.onFocusBadge,
         truncationProps = _this$props11.truncationProps,
         inputPopoverProps = _this$props11.inputPopoverProps,
         optionMatcher = _this$props11.optionMatcher,
@@ -508,12 +512,12 @@ export var EuiComboBox = /*#__PURE__*/function (_Component) {
         ariaLabelledby = _this$props11['aria-labelledby'],
         ariaDescribedby = _this$props11['aria-describedby'],
         rest = _objectWithoutProperties(_this$props11, _excluded);
-      var _this$state2 = this.state,
-        activeOptionIndex = _this$state2.activeOptionIndex,
-        hasFocus = _this$state2.hasFocus,
-        isListOpen = _this$state2.isListOpen,
-        searchValue = _this$state2.searchValue,
-        matchingOptions = _this$state2.matchingOptions;
+      var _this$state3 = this.state,
+        activeOptionIndex = _this$state3.activeOptionIndex,
+        hasFocus = _this$state3.hasFocus,
+        isListOpen = _this$state3.isListOpen,
+        searchValue = _this$state3.searchValue,
+        matchingOptions = _this$state3.matchingOptions;
 
       // Make sure we have a valid ID if users don't pass one as a prop
       var inputId = id !== null && id !== void 0 ? id : this.rootId('_eui-combobox-id');
@@ -564,7 +568,8 @@ export var EuiComboBox = /*#__PURE__*/function (_Component) {
             delimiter: delimiter,
             getSelectedOptionForSearchValue: getSelectedOptionForSearchValue,
             listboxAriaLabel: listboxAriaLabel,
-            truncationProps: truncationProps
+            truncationProps: truncationProps,
+            onFocusBadge: onFocusBadge
           });
         });
       }
@@ -589,11 +594,11 @@ export var EuiComboBox = /*#__PURE__*/function (_Component) {
             ref: _this2.comboBoxRefCallback
           }), ___EmotionJSX(EuiInputPopover, _extends({
             fullWidth: fullWidth,
-            panelPaddingSize: "none",
+            panelPaddingSize: "s",
             disableFocusTrap: true,
             closeOnScroll: true
           }, inputPopoverProps, {
-            isOpen: isListOpen,
+            isOpen: isListOpen && !noSuggestions,
             closePopover: _this2.closeList
             /* we don't want content changes to be announced via aria-live 
             because ComboBox uses a virtualized list that updates itself
@@ -601,7 +606,10 @@ export var EuiComboBox = /*#__PURE__*/function (_Component) {
             "aria-live": "off",
             input: ___EmotionJSX(EuiComboBoxInput, {
               compressed: compressed,
-              focusedOptionId: _this2.hasActiveOption() ? (_this2$state$listOpti = (_this2$state$listOpti2 = _this2.state.listOptionRefs[_this2.state.activeOptionIndex]) === null || _this2$state$listOpti2 === void 0 ? void 0 : _this2$state$listOpti2.id) !== null && _this2$state$listOpti !== void 0 ? _this2$state$listOpti : _this2.rootId("_option-".concat(_this2.state.activeOptionIndex)) : undefined,
+              focusedOptionId: _this2.hasActiveOption() ? (_this2$state$listOpti = (_this2$state$listOpti2 = _this2.state.listOptionRefs[_this2.state.activeOptionIndex]) === null || _this2$state$listOpti2 === void 0 ? void 0 : _this2$state$listOpti2.id) !== null && _this2$state$listOpti !== void 0 ? _this2$state$listOpti :
+              // uses the original `options` array to ensure a stable `id`, otherwise `aria-activedescendant`
+              // loses focus on selecting an option (due to actively removing it from the list)
+              _this2.rootId("_option-".concat(_this2.props.options.indexOf(matchingOptions[activeOptionIndex]))) : undefined,
               fullWidth: fullWidth,
               hasSelectedOptions: selectedOptions.length > 0,
               id: inputId,
@@ -791,7 +799,7 @@ EuiComboBox.propTypes = {
   "aria-labelledby": PropTypes.string,
   /**
      * By default, EuiComboBox will truncate option labels at the end of
-     * the string. You can use pass in a custom truncation configuration that
+     * the string. You can pass in a custom truncation configuration that
      * accepts any [EuiTextTruncate](/#/utilities/text-truncation) prop,
      * except for `text` and `children`.
      *