-
Notifications
You must be signed in to change notification settings - Fork 2.7k
[MM-25373] [MM-25374] [MM-25106] - Fix functionality bugs in search hint popover #5610
Conversation
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
components/search_bar/search_bar.jsx
Outdated
visibleSearchHintOptions = searchHintOptions.filter((option) => { | ||
return new RegExp(pretext, 'ig').test(option.searchTerm) && option.searchTerm.toLowerCase() !== pretext.toLowerCase(); | ||
}); | ||
} catch (err) { |
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.
is err
needed? Didn't linter complain?
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.
Whoops. Will remove
@nevyangelova |
@jgilliam17 fixed |
609c41f
to
d40d757
Compare
Thank you @nevyangelova
|
@jgilliam17 thanks, Ogi found this issue in #5485 and its fixed there. Sorry for the mash-up. |
@nevyangelova |
2669ca5
to
9fdcb37
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.
Thank you @nevyangelova
- Verified search hint popover does not display over autocomplete list
- Verified search inputs are not added inadvertently while hovering and hitting Enter
- Verified search hint popover was present when user focused on the search field.
- Verified auto-sorting as user types is working again and dash size has been increased for visibility
Issues that were raised with at-mentions and when hint list is part of the RHS will be tested after merge as it's addressed in the different PR.
Test server destroyed |
Summary
This PR addresses functionality bugs related to the search hint popover. I have tested thoroughly but let me know if you find anything else.
Ticket Link
https://mattermost.atlassian.net/browse/MM-25373
https://mattermost.atlassian.net/browse/MM-25374
https://mattermost.atlassian.net/browse/MM-25106#### Related Pull Requests