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

Revert "MM-14489 Making SuggestionList component render at bottom of input on dialogs (#2235)" #2559

Merged
merged 1 commit into from
Mar 31, 2019

Conversation

jwilander
Copy link
Member

Summary

This reverts commit ab31a59 (#2235) as it breaks both the channel switcher and other usages of the suggestion box.

See:

Screen Shot 2019-03-28 at 15 38 08
Screen Shot 2019-03-28 at 15 38 14

@jwilander jwilander added 2: Dev Review Requires review by a core commiter CherryPick/Approved Meant for the quality or patch release tracked in the milestone labels Mar 28, 2019
@jwilander jwilander added this to the v5.10.0 milestone Mar 28, 2019
@jwilander jwilander requested a review from hmhealey March 28, 2019 19:57
@jwilander
Copy link
Member Author

FYI @jasonblais @avasconcelos114 we'll need to reopen https://mattermost.atlassian.net/browse/MM-14489 and take another look at it

@jwilander jwilander merged commit 56613c6 into master Mar 31, 2019
@jwilander jwilander deleted the mm-14567 branch March 31, 2019 22:39
@jwilander jwilander added 4: Reviews Complete All reviewers have approved the pull request and removed 2: Dev Review Requires review by a core commiter labels Mar 31, 2019
jwilander added a commit that referenced this pull request Mar 31, 2019
@jwilander jwilander 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 Mar 31, 2019
@amyblais amyblais added Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation labels Apr 1, 2019
thekiiingbob pushed a commit to thekiiingbob/mattermost-webapp that referenced this pull request Apr 2, 2019
@lindy65 lindy65 added Tests/Not Needed Does not require new release tests and removed 4: Reviews Complete All reviewers have approved the pull request labels Apr 11, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
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 Tests/Not Needed Does not require new release tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants