Ensure unique ID in combobox with prepended or appended labels

CHANGELOG.md

@@ -6,6 +6,10 @@
 
 - Removed `boolean` from `EuiTour`'s `minWidth` type ([#5225](https://github.com/elastic/eui/pull/5225))
 
+**Bug fixes**
+
+Fixed missing `id` for `EuiCombobox` by generating one if `prepend` or `append` exists ([#5229](https://github.com/elastic/eui/pull/5229))
+
 ## [`38.2.0`](https://github.com/elastic/eui/tree/v38.2.0)
 
 - Updated `EuiRangeLevel` `color` property to accept CSS color values ([#5171](https://github.com/elastic/eui/pull/5171))

src-docs/src/views/combo_box/combo_box_example.js

@@ -75,15 +75,24 @@ const groupsSnippet = `<EuiComboBox
 />`;
 
 import SingleSelection from './single_selection';
+import SingleSelectionPrepend from './single_selection_prepend';
 const singleSelectionSource = require('!!raw-loader!./single_selection');
 const singleSelectionHtml = renderToHtml(SingleSelection);
+const singleSelectionPrependSource = require('!!raw-loader!./single_selection_prepend');
 const singleSelectionSnippet = `<EuiComboBox
   placeholder="Select a single option"
   singleSelection={{ asPlainText: true }}
   options={options}
   selectedOptions={selectedOptions}
   onChange={onChange}
 />`;
+const singleSelectionPrependSnippet = `<EuiComboBox
+  prepend="Prepend"
+  singleSelection={{ asPlainText: true }}
+  options={options}
+  selectedOptions={selectedOptions}
+  onChange={onChange}
+/>`;
 
 import SingleSelectionCustomOptions from './single_selection_custom_options';
 const singleSelectionCustomOptionsSource = require('!!raw-loader!./single_selection_custom_options');
@@ -406,7 +415,7 @@ export const ComboBoxExample = {
             <strong>Note:</strong> <EuiCode>append</EuiCode> and{' '}
             <EuiCode>prepend</EuiCode> props only work if
             <EuiCode>singleSelection</EuiCode> prop is not set to{' '}
-            <EuiCode>false</EuiCode> to avoid multilines that makes combobox
+            <EuiCode>false</EuiCode> to avoid multi-lines that makes combobox
             height greater than that of <EuiCode>append</EuiCode> and{' '}
             <EuiCode>prepend</EuiCode>.
           </p>
@@ -416,6 +425,34 @@ export const ComboBoxExample = {
       snippet: singleSelectionSnippet,
       demo: <SingleSelection />,
     },
+    {
+      title: 'Single selection with prepended label',
+      source: [
+        {
+          type: GuideSectionTypes.JS,
+          code: singleSelectionPrependSource,
+        },
+        {
+          type: GuideSectionTypes.HTML,
+          code: singleSelectionHtml,
+        },
+      ],
+      text: (
+        <Fragment>
+          <p>
+            <EuiCode>append</EuiCode> and <EuiCode>prepend</EuiCode> props only
+            work if
+            <EuiCode>singleSelection</EuiCode> prop is not set to{' '}
+            <EuiCode>false</EuiCode> to avoid multi-lines that makes combobox
+            height greater than that of <EuiCode>append</EuiCode> and{' '}
+            <EuiCode>prepend</EuiCode>.
+          </p>
+        </Fragment>
+      ),
+      props: { EuiComboBox, EuiComboBoxOptionOption },
+      snippet: singleSelectionPrependSnippet,
+      demo: <SingleSelectionPrepend />,
+    },
     {
       title: 'Single selection with custom options',
       source: [

src-docs/src/views/combo_box/single_selection_prepend.js

@@ -0,0 +1,57 @@
+import React, { useState } from 'react';
+
+import { EuiComboBox } from '../../../../src/components';
+
+const options = [
+  {
+    label: 'Titan',
+    'data-test-subj': 'titanOption',
+  },
+  {
+    label: 'Enceladus',
+  },
+  {
+    label: 'Mimas',
+  },
+  {
+    label: 'Dione',
+  },
+  {
+    label: 'Iapetus',
+  },
+  {
+    label: 'Phoebe',
+  },
+  {
+    label: 'Rhea',
+  },
+  {
+    label:
+      "Pandora is one of Saturn's moons, named for a Titaness of Greek mythology",
+  },
+  {
+    label: 'Tethys',
+  },
+  {
+    label: 'Hyperion',
+  },
+];
+
+export default () => {
+  const [selectedOptions, setSelected] = useState([options[2]]);
+
+  const onChange = (selectedOptions) => {
+    // We should only get back either 0 or 1 options.
+    setSelected(selectedOptions);
+  };
+
+  return (
+    <EuiComboBox
+      prepend="Prepend"
+      singleSelection={{ asPlainText: true }}
+      options={options}
+      selectedOptions={selectedOptions}
+      onChange={onChange}
+    />
+  );
+};

src/components/combo_box/__snapshots__/combo_box.test.tsx.snap

@@ -27,6 +27,7 @@ exports[`EuiComboBox is rendered 1`] = `
           <input
             aria-controls=""
             data-test-subj="comboBoxSearchInput"
+            id="generated-id__eui-combobox-id"
             role="textbox"
             style="box-sizing:content-box;width:1px"
             value=""
@@ -70,6 +71,7 @@ exports[`props autoFocus is rendered 1`] = `
     compressed={false}
     fullWidth={false}
     hasSelectedOptions={true}
+    id="generated-id__eui-combobox-id"
     inputRef={[Function]}
     isListOpen={false}
     noIcon={false}
@@ -100,6 +102,50 @@ exports[`props autoFocus is rendered 1`] = `
 </div>
 `;
 
+exports[`props custom ID is rendered 1`] = `
+<div
+  aria-expanded={false}
+  aria-haspopup="listbox"
+  className="euiComboBox"
+  onKeyDown={[Function]}
+  role="combobox"
+>
+  <EuiComboBoxInput
+    autoSizeInputRef={[Function]}
+    compressed={false}
+    fullWidth={false}
+    hasSelectedOptions={true}
+    id="test-id-1"
+    inputRef={[Function]}
+    isListOpen={false}
+    noIcon={false}
+    onChange={[Function]}
+    onClear={[Function]}
+    onClick={[Function]}
+    onCloseListClick={[Function]}
+    onFocus={[Function]}
+    onOpenListClick={[Function]}
+    onRemoveOption={[Function]}
+    rootId={[Function]}
+    searchValue=""
+    selectedOptions={
+      Array [
+        Object {
+          "label": "Mimas",
+        },
+        Object {
+          "label": "Iapetus",
+        },
+      ]
+    }
+    singleSelection={false}
+    toggleButtonRef={[Function]}
+    updatePosition={[Function]}
+    value="Mimas, Iapetus"
+  />
+</div>
+`;
+
 exports[`props delimiter is rendered 1`] = `
 <div
   aria-expanded={false}
@@ -113,6 +159,7 @@ exports[`props delimiter is rendered 1`] = `
     compressed={false}
     fullWidth={false}
     hasSelectedOptions={true}
+    id="generated-id__eui-combobox-id"
     inputRef={[Function]}
     isListOpen={false}
     noIcon={false}
@@ -156,6 +203,7 @@ exports[`props full width is rendered 1`] = `
     compressed={false}
     fullWidth={true}
     hasSelectedOptions={true}
+    id="generated-id__eui-combobox-id"
     inputRef={[Function]}
     isListOpen={false}
     noIcon={false}
@@ -196,6 +244,7 @@ exports[`props isClearable=false disallows user from clearing input when no opti
     compressed={false}
     fullWidth={false}
     hasSelectedOptions={false}
+    id="generated-id__eui-combobox-id"
     inputRef={[Function]}
     isListOpen={false}
     noIcon={false}
@@ -229,6 +278,7 @@ exports[`props isClearable=false disallows user from clearing input when options
     compressed={false}
     fullWidth={false}
     hasSelectedOptions={true}
+    id="generated-id__eui-combobox-id"
     inputRef={[Function]}
     isListOpen={false}
     noIcon={false}
@@ -271,6 +321,7 @@ exports[`props isDisabled is rendered 1`] = `
     compressed={false}
     fullWidth={false}
     hasSelectedOptions={true}
+    id="generated-id__eui-combobox-id"
     inputRef={[Function]}
     isDisabled={true}
     isListOpen={false}
@@ -324,6 +375,7 @@ exports[`props options list is rendered 1`] = `
           <input
             aria-controls="generated-id_listbox"
             data-test-subj="comboBoxSearchInput"
+            id="generated-id__eui-combobox-id"
             role="textbox"
             style="box-sizing: content-box; width: 2px;"
             value=""
@@ -632,6 +684,7 @@ exports[`props selectedOptions are rendered 1`] = `
     compressed={false}
     fullWidth={false}
     hasSelectedOptions={true}
+    id="generated-id__eui-combobox-id"
     inputRef={[Function]}
     isListOpen={false}
     noIcon={false}
@@ -675,6 +728,7 @@ exports[`props singleSelection is rendered 1`] = `
     compressed={false}
     fullWidth={false}
     hasSelectedOptions={true}
+    id="generated-id__eui-combobox-id"
     inputRef={[Function]}
     isListOpen={false}
     noIcon={false}
@@ -706,7 +760,7 @@ exports[`props singleSelection prepend and append is rendered 1`] = `
 <div
   aria-expanded={true}
   aria-haspopup="listbox"
-  className="euiComboBox euiComboBox-isOpen"
+  className="euiComboBox euiComboBox--prepended euiComboBox--appended euiComboBox-isOpen"
   onKeyDown={[Function]}
   role="combobox"
 >
@@ -716,6 +770,7 @@ exports[`props singleSelection prepend and append is rendered 1`] = `
     compressed={false}
     fullWidth={false}
     hasSelectedOptions={false}
+    id="generated-id__eui-combobox-id"
     inputRef={[Function]}
     isListOpen={true}
     noIcon={false}
@@ -845,6 +900,7 @@ exports[`props singleSelection selects existing option when opened 1`] = `
     compressed={false}
     fullWidth={false}
     hasSelectedOptions={true}
+    id="generated-id__eui-combobox-id"
     inputRef={[Function]}
     isListOpen={true}
     noIcon={false}

src/components/combo_box/combo_box.test.tsx

@@ -92,6 +92,18 @@ describe('props', () => {
     expect(component).toMatchSnapshot();
   });
 
+  test('custom ID is rendered', () => {
+    const component = shallow(
+      <EuiComboBox
+        id="test-id-1"
+        options={options}
+        selectedOptions={[options[2], options[4]]}
+      />
+    );
+
+    expect(component).toMatchSnapshot();
+  });
+
   describe('isClearable=false disallows user from clearing input', () => {
     test('when no options are selected', () => {
       const component = shallow(

src/components/combo_box/combo_box.tsx

@@ -936,6 +936,9 @@ export class EuiComboBox<T> extends Component<
       matchingOptions,
     } = this.state;
 
+    // Make sure we have a valid ID if users don't pass one as a prop
+    const inputId = id || this.rootId('_eui-combobox-id');
+
     // Visually indicate the combobox is in an invalid state if it has lost focus but there is text entered in the input.
     // When custom options are disabled and the user leaves the combo box after entering text that does not match any
     // options, this tells the user that they've entered invalid input.
@@ -946,6 +949,8 @@ export class EuiComboBox<T> extends Component<
     const classes = classNames('euiComboBox', className, {
       'euiComboBox--compressed': compressed,
       'euiComboBox--fullWidth': fullWidth,
+      'euiComboBox--prepended': prepend,
+      'euiComboBox--appended': append,
       'euiComboBox-isDisabled': isDisabled,
       'euiComboBox-isInvalid': markAsInvalid,
       'euiComboBox-isOpen': isListOpen,
@@ -1030,7 +1035,7 @@ export class EuiComboBox<T> extends Component<
           }
           fullWidth={fullWidth}
           hasSelectedOptions={selectedOptions.length > 0}
-          id={id}
+          id={inputId}
           inputRef={this.searchInputRefCallback}
           isDisabled={isDisabled}
           isListOpen={isListOpen}

src/components/combo_box/combo_box_input/combo_box_input.tsx

@@ -264,6 +264,7 @@ export class EuiComboBoxInput<T> extends Component<
       <EuiFormControlLayout
         icon={icon}
         {...clickProps}
+        inputId={id}
         isLoading={isLoading}
         compressed={compressed}
         fullWidth={fullWidth}

src/themes/eui-amsterdam/overrides/_form_control_layout.scss

@@ -1,3 +1,19 @@
+.euiComboBox {
+  // Round the left border when we append a label
+  &--appended {
+    .euiFormControlLayout__childrenWrapper {
+      border-radius: $euiFormControlBorderRadius 0 0 $euiFormControlBorderRadius;
+    }
+  }
+
+  // Round the right border when we prepend a label
+  &--prepended {
+    .euiFormControlLayout__childrenWrapper {
+      border-radius: 0 $euiFormControlBorderRadius $euiFormControlBorderRadius 0;
+    }
+  }
+}
+
 .euiFormControlLayout--group {
   border-radius: $euiFormControlBorderRadius;
   background-color: $euiFormInputGroupLabelBackground;