Skip to content
This repository has been archived by the owner on Mar 13, 2024. It is now read-only.

MM-25492, MM-25485, MM-25483 - Autocomplete ui fixes #5593

Merged
merged 3 commits into from
May 27, 2020
Merged

MM-25492, MM-25485, MM-25483 - Autocomplete ui fixes #5593

merged 3 commits into from
May 27, 2020

Conversation

asaadmahmood
Copy link
Contributor

@asaadmahmood asaadmahmood commented May 26, 2020

Summary

Autocomplete ui fixes
MM-25492 - Aligning autocomplete to the left
MM-25485 - Updating icon sizes in switcher modal
MM-25483 - Updating icon sizes in popover

Ticket Link

https://mattermost.atlassian.net/browse/MM-25492
https://mattermost.atlassian.net/browse/MM-25485
https://mattermost.atlassian.net/browse/MM-25483

Screenshots

Screenshot 2020-05-27 at 1 36 27 AM
Screenshot 2020-05-27 at 1 36 40 AM
Screenshot 2020-05-27 at 1 36 52 AM

@asaadmahmood asaadmahmood added 2: Dev Review Requires review by a core commiter 3: QA Review Requires review by a QA tester CherryPick/Approved Meant for the quality or patch release tracked in the milestone AutomatedCherryPick labels May 26, 2020
@asaadmahmood asaadmahmood added this to the v5.24.0 milestone May 26, 2020
@asaadmahmood
Copy link
Contributor Author

@lindy65 This may not fix your issue of the bottom border not showing as that's an edge case for a very small screen height.

@lindy65 lindy65 added the Setup Cloud Test Server Setup a test server using Mattermost Cloud label May 27, 2020
Copy link
Contributor

@lindy65 lindy65 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @asaadmahmood - issues listed in all 3 tickets are fixed. 👍

As you mentioned above, the issue of the bottom border is not fixed for me in my normal view but, if I make the text size smaller on my screen, then I am able to see the bottom border.

@lindy65 lindy65 added QA Review Done and removed 3: QA Review Requires review by a QA tester Setup Cloud Test Server Setup a test server using Mattermost Cloud labels May 27, 2020
@mm-cloud-bot
Copy link

Test server destroyed

@cpoile cpoile added 4: Reviews Complete All reviewers have approved the pull request and removed 2: Dev Review Requires review by a core commiter labels May 27, 2020
@cpoile cpoile merged commit dedd821 into mattermost:master May 27, 2020
@mattermod mattermod added CherryPick/Done Successfully cherry-picked to the quality or patch release tracked in the milestone and removed CherryPick/Approved Meant for the quality or patch release tracked in the milestone labels May 27, 2020
mattermod pushed a commit that referenced this pull request May 27, 2020
@asaadmahmood asaadmahmood deleted the autocomplete-ui-fixes branch May 27, 2020 14:33
@amyblais amyblais added Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation labels May 28, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
4: Reviews Complete All reviewers have approved the pull request AutomatedCherryPick Changelog/Not Needed Does not require a changelog entry CherryPick/Done Successfully cherry-picked to the quality or patch release tracked in the milestone Docs/Not Needed Does not require documentation QA Review Done
Projects
None yet
7 participants