Skip to content
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

minor fix - collapse button hover and dropdown button width #7748

Merged
merged 1 commit into from
Oct 16, 2024

Conversation

ehconitin
Copy link
Contributor

follow up - #7414

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR Summary

This pull request makes minor adjustments to the MultiWorkspaceDropdownButton and NavigationDrawerCollapseButton components, improving their appearance and behavior in the navigation drawer.

  • Modified MultiWorkspaceDropdownButton.tsx for responsive styling and better content alignment
  • Adjusted NavigationDrawerCollapseButton.tsx dimensions from 5x6 to 4x4 spacing units
  • Removed hover effect from NavigationDrawerCollapseButton for a cleaner appearance
  • Changes aim to enhance UI consistency and responsiveness in both expanded and collapsed states
  • Related to issue feat - Compact sidebar #7414 for implementing a compact sidebar feature

2 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile

@bosiraphael bosiraphael self-assigned this Oct 16, 2024
Copy link
Contributor

@bosiraphael bosiraphael left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM :)

@bosiraphael bosiraphael merged commit fad0414 into twentyhq:main Oct 16, 2024
11 checks passed
Copy link

Thanks @ehconitin for your contribution!
This marks your 49th PR on the repo. You're top 1% of all our contributors 🎉
See contributor page - Share on LinkedIn - Share on Twitter

Contributions

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants