-
Notifications
You must be signed in to change notification settings - Fork 24
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
PORTALS-2651 - react-select components #1546
Conversation
0965b96
to
9358ad9
Compare
- Update UserSearchBoxV2, QuerySortSelector, AccessRequirementSearchBox to no longer use bootstrap classes - Remove UserSearchBox v1
9358ad9
to
5bf311f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unused
) | ||
}, | ||
} | ||
|
||
export default function AccessRequirementSearchBox( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -181,8 +174,7 @@ function UserSearchBoxV2(props: UserSearchBoxProps) { | |||
isClearable | |||
value={value} | |||
styles={{ | |||
// Bootstrap's form-control class overrides the display value, manually set to flex (the default without Bootstrap) | |||
control: styles => ({ ...styles, display: 'flex !important' }), | |||
control: styles => ({ ...styles, backgroundColor: palette.grey[200] }), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
</components.Control> | ||
) | ||
} | ||
|
||
function QuerySortSelector({ sortConfig }: QuerySortSelectorProps) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove some bootstrap-specific classes