-
Notifications
You must be signed in to change notification settings - Fork 2.7k
[MM-22815][MM-23693][MM-23735] - Improve empty states on RHS search variants #5485
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.
Nice @nevyangelova! Just a question about FormattedHTMLMessage
.
import NoResultsIndicator from 'components/no_results_indicator/no_results_indicator.tsx'; | ||
import FlagIcon from 'components/widgets/icons/flag_icon'; | ||
|
||
import {NoResultsVariant} from '../no_results_indicator/types'; |
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.
import {NoResultsVariant} from '../no_results_indicator/types'; | |
import {NoResultsVariant} from 'components/no_results_indicator/types'; |
width: 11px; | ||
margin: 2px; | ||
vertical-align: middle; | ||
} |
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.
nit: empty line
@nevyangelova I tried spinning up a test server, but looks like it failed. I'll review when this is working again. |
/update-branch |
@matthewbirtch test server finally succeeded :D |
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.
@nevyangelova I'm seeing a few issues:
-
Mentions empty state not replaced with Search empty state:
- Click on Recent Mentions (@ icon)
- Enter something in search that won't return any results and hit enter
Expected: Search empty state is shown
Observed: Mentions empty state is shown
- Recent Mentions icon in active state after searching
/update-branch |
@nevyangelova
|
@nevyangelova @matthewbirtch I noticed that empty states for mentions, pinned and flagged have added |
@jgilliam17 this PR is approved by UX. |
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
Tested, looks good to merge.
- Verified emoji empty state
- Verified empty state for mentions, flagged and pinned
- Verified search empty state
Per discussion with Nevy, issue with search results displaying on the RHS after user clears the at-mention search can be addressed separately after merge and testing as it’s out of scope here.
Test server destroyed |
Summary
As part of the UI quick wins, this PR handles empty states in the RHS search variants. Included is "mentions", "search", "flagged posts", "pinned posts" and "emoji picker". It add the "no_results_indicator.tsx" component adapted to match the respective designs.
Ticket Link
https://mattermost.atlassian.net/browse/MM-23693
https://mattermost.atlassian.net/browse/MM-22815
https://mattermost.atlassian.net/browse/MM-23735