From 66bc3f8e988adcc106475a6b7984ba6930f5d7b8 Mon Sep 17 00:00:00 2001 From: Carlos Martins Date: Wed, 25 Jan 2023 15:54:44 -0700 Subject: [PATCH] revert changes and disable highlight if option is disabled --- src/components/OptionsList/BaseOptionsList.js | 4 +++- src/components/OptionsSelector/BaseOptionsSelector.js | 2 +- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/src/components/OptionsList/BaseOptionsList.js b/src/components/OptionsList/BaseOptionsList.js index b7b6ebcd5a41..781b063f1cc6 100644 --- a/src/components/OptionsList/BaseOptionsList.js +++ b/src/components/OptionsList/BaseOptionsList.js @@ -154,18 +154,20 @@ class BaseOptionsList extends Component { * @return {Component} */ renderItem({item, index, section}) { + const isDisabled = this.props.isDisabled || section.isDisabled; return ( option.login === item.login))} showSelectedState={this.props.canSelectMultipleOptions} boldStyle={this.props.boldStyle} - isDisabled={this.props.isDisabled || section.isDisabled} + isDisabled={isDisabled} shouldHaveOptionSeparator={index > 0 && this.props.shouldHaveOptionSeparator} /> ); diff --git a/src/components/OptionsSelector/BaseOptionsSelector.js b/src/components/OptionsSelector/BaseOptionsSelector.js index 88ed99284d48..c473c426d80f 100755 --- a/src/components/OptionsSelector/BaseOptionsSelector.js +++ b/src/components/OptionsSelector/BaseOptionsSelector.js @@ -101,7 +101,7 @@ class BaseOptionsSelector extends Component { } const newOptions = this.flattenSections(); - const newFocusedIndex = this.props.shouldTextInputAppearBelowOptions ? newOptions.length : this.props.selectedOptions.length; + const newFocusedIndex = this.props.selectedOptions.length; // eslint-disable-next-line react/no-did-update-set-state this.setState({ allOptions: newOptions,