-
Notifications
You must be signed in to change notification settings - Fork 50
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: No Scrolling Required for Some TLDR
Descriptions Due to Short Length
#1318
Fix: No Scrolling Required for Some TLDR
Descriptions Due to Short Length
#1318
Conversation
Hi @Rassl, |
@MuhammadUmer44 fix conflicts please |
@Rassl, Ok i fix it and ping you. |
Done |
@MuhammadUmer44 I don't see any reason to use js calculation for this, it should be pure css fix, or not a bug at all. |
@Rassl, please see the live website: https://second-brain.sphinx.chat/ |
Hi @Rassl, |
Hi @Rassl, |
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
Problem:
TLDR
descriptions do not require scrolling because they are short and do not extend beyond the visible area. However, the scrolling functionality is still enabled, leading to unnecessary user interaction and potentially affecting the user experience.Expected Behavior:
closes: #1287
Issue ticket number and link:
TLDR
Descriptions Due to Short Length #1287 ]Evidence:
https://www.loom.com/share/44558d3f1257469987d70fcc6079b360