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

fix x axis scroll bar issue on developer page #7975

Merged
merged 1 commit into from
Oct 23, 2024

Conversation

sans-byte
Copy link
Contributor

This PR Fixes Issue : #7932
I have added the CSS to remove the scroll bar from x axis of the sidebar on the developers page.

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 PR addresses the horizontal scrollbar issue on the developer page by modifying the DocsSideBar.tsx file.

  • Added overflow-x: hidden to StyledContainer in DocsSideBar.tsx to hide the horizontal scrollbar
  • Maintains vertical scrolling functionality while resolving the issue
  • Directly addresses the problem described in issue [🕹️] bug report vertical scrollbar in https://twenty.com/developers #7932
  • Implements a simple and effective solution without introducing new complications
  • Aligns with the expected behavior outlined in the issue, improving the sidebar's design

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

Copy link
Member

@FelixMalfait FelixMalfait left a comment

Choose a reason for hiding this comment

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

I tried to test with Microsoft Edge on Mac but can't reproduce. I think it's okay. Thanks!

@FelixMalfait FelixMalfait merged commit 74ecacb into twentyhq:main Oct 23, 2024
2 checks passed
@FelixMalfait
Copy link
Member

/award 50

Copy link

oss-gg bot commented Oct 23, 2024

Awarding sans-byte: 50 points 🕹️ Well done! Check out your new contribution on oss.gg/sans-byte

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