segmentio/evergreen

Combobox `autocompleteProps` should be `Omit<AutocompleteProps, 'children' | 'items' | 'onChange'>`

brandongregoryscott opened this issue · 0 comments

When providing autocompleteProps to the Combobox, it requires items, onChange, and children, which don't really make sense - the Combobox component manages these internally. It looks like the Combobox spreads these props after the items/onChange from its own props, so it won't "break" anything by allowing these thru, but it's not really the intended API of the component.

See this CodeSandbox for an example of passing items and onChange through autocompleteProps: https://codesandbox.io/s/combobox-autocompleteprops-override-dc4rle?file=/index.js