-
Notifications
You must be signed in to change notification settings - Fork 434
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
follow-up(ChatView) - align scroll button position with mobile clients #9861
Conversation
Signed-off-by: Maksim Sukharev <[email protected]>
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.
I think the button should be within the boundaries of the message list, with 8px bottom margin to the new message form component
You mean to lift it up on 8px, not lower? For now I restored it's initial position, so it will be a design change. |
Icon change sounds good. I mean exactly like the screenshot you posted, ~ 8px above the |
Signed-off-by: Maksim Sukharev <[email protected]>
/backport to stable27 |
☑️ Resolves
🖼️ Screenshots
🚧 Tasks
🏁 Checklist
docs/
has been updated or is not required