fix: reset highlightedIndex to defaultHighlightIndex after selecting …

…item (#195)
downshift-js · Sep 21, 2017 · efbbe19 · efbbe19
1 parent d48031f
commit efbbe19
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/src/__tests__/downshift.get-item-props.js b/src/__tests__/downshift.get-item-props.js
@@ -50,6 +50,19 @@ test('on mouseenter of an item updates the highlightedIndex to that item', () =>
   )
 })
 
+test('after selecting an item highlightedIndex should be reset to defaultHighlightIndex', () => {
+  const {Component, childSpy} = setup()
+  const wrapper = mount(<Component defaultHighlightedIndex={1} />)
+  const firstButton = wrapper.find('button').first()
+  childSpy.mockClear()
+  firstButton.simulate('click')
+  expect(childSpy).toHaveBeenCalledWith(
+    expect.objectContaining({
+      highlightedIndex: 1,
+    }),
+  )
+})
+
 test('getItemProps throws a helpful error when no object is given', () => {
   expect(() =>
     mount(

diff --git a/src/downshift.js b/src/downshift.js
@@ -217,7 +217,7 @@ class Downshift extends Component {
     this.internalSetState(
       {
         isOpen: false,
-        highlightedIndex: null,
+        highlightedIndex: this.props.defaultHighlightedIndex,
         selectedItem: item,
         inputValue: this.props.itemToString(item),
         ...otherStateToSet,