Krishna
Krishna
@dangrous, All checks have passed, you can merge after final reviewing.
@dangrous ## Explanation for 1239,1240. On the line below, we use `getFilteredOptions` with `emptyCategoriesList` and `selectedOptions`. Previously, `emptySelectedResultList` was returning `isDisabled: false` & `isSelected: false` for the Medical category (which...
### After fix: | Everything Selected | Workspace Selected | | ------------- | ------------- | |  |  | cc: @Expensify/design
In the design the input placeholder is `Find a workspace` and there is a search icon on the left of the input, do we need that as well?
> can you try that and show us what it looks like? @shawnborton @getusha, IMO, we shouldn't add this because no other input in the app has this icon, or...
@dannymcclain @shawnborton, currently the search icon has 8px padding right, if we remove the padding right from the icon it will still not look aligned with above 2 icons, so...
@getusha, I think we don't want to make any changes here, can you pls start reviewing.
@getusha, including the Everything option as a selectable using keyboard will be tricky, can we pls confirm if that is required. cc: @shawnborton @rlinoz
@dannymcclain @shawnborton thanks! @getusha, you can continue the review.
@getusha, bump for review.