-
Notifications
You must be signed in to change notification settings - Fork 2.7k
[MM-23414] Fix up multiselect styling #5087
[MM-23414] Fix up multiselect styling #5087
Conversation
e03ab55
to
b4c955e
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.
See comment. And I think we can keep the react-select class and react-select-auto class.
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.
LGTM
@asaadmahmood Thanks - I didnt realize you already had a fix for this - I merged my changes with yours and kept the styling changes I made |
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.
Awesome, thank you.
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.
Tested with the four in-app default themes, two of the custom themes from https://docs.mattermost.com/help/settings/theme-colors.html#custom-theme-examples, and using the color pickers to create a new custom theme. No issues found on that selector or others that I looked at, including the invite people UI as mentioned above. LGTM!
Test server destroyed |
Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour. |
@fm2munsh
|
* Improve direct messages multiselect styling * Re-add react-select class * Extra quote
Summary
/components/widgets/inputs/user_emails_input.scss
(from the invite team members select list)Ticket Link
https://mattermost.atlassian.net/browse/MM-23414
Before
After