-
-
Notifications
You must be signed in to change notification settings - Fork 59
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
fix(chat): added ellipsis for long chat names #534
fix(chat): added ellipsis for long chat names #534
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
can we use |
It's worth a try, but it requires changing the markup structure, because the chat name and the date are not placed in the same div. |
@juliahermak Fix the date position. Compare with the version on msg.adamant.im date-position-bug.mov |
@bludnic fixed |
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.
Looks good.
@martiliones I want to see the deployment build to see how it looks like. |
@adamant-al @bludnic how do we force vercel to redeploy? |
@adamant-al vercel actually deployed the commit but didn't show it https://adamant-8mccwsuyo-adamant-team.vercel.app/ see https://github.com/Adamant-im/adamant-im/deployments (surge's one is failed) |
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.
Looks good, both on Chrome and Firefox.
Merge it.
Resolves https://trello.com/c/yRtWY0uF/433-ui-reduce-name-field-size-in-the-chat-list-screen