-
Notifications
You must be signed in to change notification settings - Fork 683
feat(ui): make codegen input scrollable with and actions overlay #10250
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
feat(ui): make codegen input scrollable with and actions overlay #10250
Conversation
Move to a borderless textarea wrapper and add maxRows so the input can grow up to 8 lines before scrolling. Constrain the input container with overflow hidden and a fixed bottom padding to make space for action buttons. Add a semi-opaque fade overlay at the bottom to visually mask the scroll boundary, and place the action toolbar above the fade using z-index so buttons remain accessible. Also add a subtle border, radius, and background to match other dialogs.
@SilkePilon is attempting to deploy a commit to the GitButler Team on Vercel. A member of the Team first needs to authorize it. |
Preview: Recording.2025-09-12.151426.mp4I also changed the border radius to be the same as the other ui elements, tho this can be reverted |
Thank you very much for the fix. |
That was a different issue. It was always on the right and blocked the text. |
@SilkePilon I'll make some changes |
- smaller gradient height - smaller border radius
@SilkePilon I can merge it if you okay with these changes |
👍 Looks good to me! |
@krlvi @SilkePilon there is an issue with the scrollbar ![]() ![]() This could mean that we need to treat the input differently, avoiding the use of inner textarea padding. I’ll address this in the next PR. I see if I can save this nice gradient without adding extra JS logic |
Move to a borderless textarea wrapper and add maxRows so the input can grow up to 8 lines before scrolling. Constrain the input container with overflow hidden and a fixed bottom padding to make space for action buttons. Add a semi-opaque fade overlay at the bottom to visually mask the scroll boundary, and place the action toolbar above the fade using z-index so buttons remain accessible. Also add a subtle border, radius, and background to match other dialogs.