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: Title overflows in mobile viewport for right drawer #7311

Merged
merged 3 commits into from
Sep 30, 2024

Conversation

harshit078
Copy link
Contributor

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 issue of title overflow in the mobile viewport for the right drawer component. The change improves the user experience on mobile devices by preventing horizontal scrolling caused by long titles.

  • Added overflow-x: hidden to StyledInfoContainer for mobile viewports
  • Imported MOBILE_VIEWPORT constant from 'twenty-ui' for consistent breakpoint usage
  • Applied the fix specifically to the ShowPageSummaryCard component, which is used in the right drawer
  • The change affects the layout of notes and tasks where the rightDrawer component is used

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

Copy link
Contributor

@martmull martmull left a comment

Choose a reason for hiding this comment

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

Thanks you for your contribution, I suggest a simplification

@martmull martmull merged commit 95e1053 into twentyhq:main Sep 30, 2024
4 of 8 checks passed
Copy link

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

Contributions

harshit078 added a commit to harshit078/twenty that referenced this pull request Oct 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants