From 99f018faac9958b0a443dd41685e5cde2e01174e Mon Sep 17 00:00:00 2001 From: Ashish Bhate Date: Fri, 25 Sep 2020 13:43:19 +0530 Subject: [PATCH] [MM-28706] revert ordering change for `in:` autosuggest search (#4818) Summary: Revert ordering change for in: auto-suggest search after user feedback. The ordering now is Public Channels, Private Channels, and finally DMs and GMs. This PR only fixes ordering. Further search improvements are planned in https://mattermost.atlassian.net/browse/MM-26805 Ticket Link: https://mattermost.atlassian.net/browse/MM-28706 The Jira ticket above reverts the changes in https://mattermost.atlassian.net/browse/MM-27540 (Parent: https://mattermost.atlassian.net/browse/MM-15477) which was implemented in #4704 --- .../channel_mention/channel_mention.js | 22 +++++++++---------- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/app/components/autocomplete/channel_mention/channel_mention.js b/app/components/autocomplete/channel_mention/channel_mention.js index 639ecbb65..e72509682 100644 --- a/app/components/autocomplete/channel_mention/channel_mention.js +++ b/app/components/autocomplete/channel_mention/channel_mention.js @@ -91,12 +91,13 @@ export default class ChannelMention extends PureComponent { myMembers !== this.props.myMembers)) { const sections = []; if (isSearch) { - if (directAndGroupMessages.length) { + if (publicChannels.length) { sections.push({ - id: t('suggestion.search.direct'), - defaultMessage: 'Direct Messages', - data: directAndGroupMessages, - key: 'directAndGroupMessages', + id: t('suggestion.search.public'), + defaultMessage: 'Public Channels', + data: publicChannels.filter((cId) => myMembers[cId]), + key: 'publicChannels', + hideLoadingIndicator: true, }); } @@ -110,13 +111,12 @@ export default class ChannelMention extends PureComponent { }); } - if (publicChannels.length) { + if (directAndGroupMessages.length) { sections.push({ - id: t('suggestion.search.public'), - defaultMessage: 'Public Channels', - data: publicChannels.filter((cId) => myMembers[cId]), - key: 'publicChannels', - hideLoadingIndicator: true, + id: t('suggestion.search.direct'), + defaultMessage: 'Direct Messages', + data: directAndGroupMessages, + key: 'directAndGroupMessages', }); } } else {