-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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 LeftPanel Skeleton Alignment #5707
Conversation
- remove the extra skeletons - use the exact flex gap structuring as used by the original items.
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.
PR Summary
- Adjusted flex gap in
StyledSkeletonContainer
from 32px to 12px - Introduced
MainNavigationDrawerContainer
with a 32px gap - Removed extra skeleton loaders to streamline layout
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.
Hey, thanks for your changes, left some comments
@@ -26,7 +26,7 @@ const StyledItemsContainer = styled.div` | |||
const StyledSkeletonContainer = styled.div` | |||
display: flex; | |||
flex-direction: column; | |||
gap: 32px; | |||
gap: 12px; |
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.
${({ theme }) => theme.spacing(3)}
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.
please, use theme.spacing
utils everywhere in this file
const MainNavigationDrawerContainer = styled.div` | ||
display: flex; | ||
flex-direction: column; | ||
gap: 32px; |
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.
${({ theme }) => theme.spacing(8)}
Closing as duplicate of #5705 |
fixes #5664.
Result
Screen.Recording.2024-06-02.at.1.54.03.AM.mov