-
Notifications
You must be signed in to change notification settings - Fork 8.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
UX: add illustrations for empty chat list + split into tabs on drawer #26910
Conversation
a140fb4
to
51dd237
Compare
…scourse/discourse into chat-illustrations-empty-list
I like the splash image for empty channel lists, very nice.. I wonder if we need the "0" though? Not sure if it's just me but feels a little strange having it there on top of the graphic. |
plugins/chat/assets/javascripts/discourse/components/channels-list-public.gjs
Outdated
Show resolved
Hide resolved
Just a reminder in case I forget later, I think we need to fix this UX issue first where channels and DMs are split |
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 👍
Co-authored-by: Régis Hanol <regis@hanol.fr>
This pull request has been mentioned on Discourse Meta. There might be relevant details there: https://meta.discourse.org/t/new-strings-from-discourse-discourse-arent-added-to-crowdin/309802/1 |
Juicing Discourse by adding a placeholder image + CTA in chat, for empty channel and DM lists.
Drawer
Mobile
Desktop (header dropdown navigation)