- removed redundant open flag

- reordered handlers and render function

Author: bmdalex

docs/src/examples/components/Dropdown/Usage/DropdownExampleControlled.shorthand.tsx

@@ -6,6 +6,10 @@ const inputItems = ['Bruce Wayne', 'Natasha Romanoff', 'Steven Strange', 'Alfred
 class DropdownExampleControlled extends React.Component {
   state = { open: false }
 
+  handleOpenChange = (e, { open }) => {
+    this.setState({ open })
+  }
+
   render() {
     const open = this.state.open
     return (
@@ -20,10 +24,6 @@ class DropdownExampleControlled extends React.Component {
       </Flex>
     )
   }
-
-  handleOpenChange = (e, { open }) => {
-    this.setState({ open })
-  }
 }
 
 export default DropdownExampleControlled

packages/react/src/components/Dropdown/Dropdown.tsx

@@ -384,7 +384,6 @@ class Dropdown extends AutoControlledComponent<Extendable<DropdownProps>, Dropdo
                   {this.renderItemsList(
                     styles,
                     variables,
-                    open,
                     highlightedIndex,
                     toggleMenu,
                     selectItemAtIndex,
@@ -475,7 +474,6 @@ class Dropdown extends AutoControlledComponent<Extendable<DropdownProps>, Dropdo
   private renderItemsList(
     styles: ComponentSlotStylesInput,
     variables: ComponentVariablesInput,
-    open: boolean,
     highlightedIndex: number,
     toggleMenu: () => void,
     selectItemAtIndex: (index: number) => void,
@@ -484,6 +482,7 @@ class Dropdown extends AutoControlledComponent<Extendable<DropdownProps>, Dropdo
     getInputProps: (options?: GetInputPropsOptions) => any,
   ) {
     const { search } = this.props
+    const { open } = this.state
     const { innerRef, ...accessibilityMenuProps } = getMenuProps(
       { refKey: 'innerRef' },
       { suppressRefError: true },