-
Notifications
You must be signed in to change notification settings - Fork 376
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
[BUG] Scrollbar effect #536
Comments
It's great having you contribute to this project by creating an issueMake sure you are assigned to this before you work on it and you read the Contributing.md file, Thank you! Welcome to the community 🤓 |
Hello @Dun-sin. I can work on this issue. But just want to ask if we should just remove scrollbar display by keeping scrolling on overflow or show scrollbar only when the content overflows the container. |
Second one is a better choice. You've got it. thanks for wanting to contribute. Follow the rules here, or your PR won't be accepted and will be closed. Good luck! |
Yesterday I was not getting the error. Now i forked the latest code from your repo and started getting this error. |
I understand, so you need to add |
Yes now working perfectly. I tested changes done by me and they are working. After the messages overflow the container then only the scrollbar appear. So I don't need to push anymore changes. Thank you. |
Description
in chat the scrollbar behaviour is weird looking
Screenshots
Additional information
No response
🥦 Browser
Google Chrome
👀 Have you checked if this issue has been raised before?
🏢 Have you read the Contributing Guidelines?
Are you willing to work on this issue ?
None
The text was updated successfully, but these errors were encountered: