-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
refactor(ui): Fix scrolling behavior for compact entity profile #7599
refactor(ui): Fix scrolling behavior for compact entity profile #7599
Conversation
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.
lookin good
@@ -73,6 +73,10 @@ type Props<T, U> = { | |||
isNameEditable?: boolean; | |||
}; | |||
|
|||
const MAX_SIDEBAR_WIDTH = 800; | |||
const MIN_SIDEBAR_WIDTH = 200; | |||
const MAX_COMPACT_WIDTH = 490 - 24 * 2; |
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.
where does this number come from?
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.
we give the slide out modal a max width of 490 elsewhere :/
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Summary
Previously, there was horizontal scroll in the compact entity profile shown in lineage when there were long browse paths or other content. This PR fixes it by getting rid of horizontal scroll.
Status
Ready for review
Screenshots
Before ->
After ->
Checklist