Skip to content

fix (admin & sdk): text-area out of bounds when resized#1452

Open
paanSinghCoder wants to merge 1 commit intomainfrom
fix/textarea-out-of-bounds
Open

fix (admin & sdk): text-area out of bounds when resized#1452
paanSinghCoder wants to merge 1 commit intomainfrom
fix/textarea-out-of-bounds

Conversation

@paanSinghCoder
Copy link
Contributor

@paanSinghCoder paanSinghCoder commented Mar 17, 2026

Summary

fix (admin & sdk): email text-area out of bounds on resize

  • Allow only vertical resize for the user invite text-area.
  • Set width as 100% to avoid going out of container modal.
  • Set max height as var(--rs-space-17) (120px).

Routes affected:

  • Admin - /organizations/{org-id}/members -> Three dots -> Invite User... dialog.
  • SDK - Members Tab -> Invite User
sdk1.mov
sdk2.mov

Changes

Add className .invite-users-emails-textarea to the email text-area with below CSS:

.invite-users-emails-textarea {
  resize: vertical;
  width: 100%;
  max-height: var(--rs-space-17);
}

Test Plan

  • Manual testing completed
  • Build and type checking passes

@paanSinghCoder paanSinghCoder self-assigned this Mar 17, 2026
@vercel
Copy link

vercel bot commented Mar 17, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
frontier Ready Ready Preview, Comment Mar 17, 2026 7:26am

@coderabbitai
Copy link

coderabbitai bot commented Mar 17, 2026

📝 Walkthrough

Summary by CodeRabbit

  • Style
    • Improved email textarea styling in invitation dialogs. Email input fields in user and member invitation workflows now feature vertical-only resizing with full-width display and optimized maximum height constraints. These changes enhance visual consistency, provide better form organization, and improve the overall user experience when managing invitations across admin and member management sections.

Walkthrough

This PR adds CSS styling to TextArea components in two invite dialog interfaces. It applies a new className to email input fields and defines corresponding CSS classes to control textarea resizing and dimensions using CSS variables.

Changes

Cohort / File(s) Summary
Email TextArea CSS Styling
web/sdk/admin/views/organizations/details/layout/layout.module.css, web/sdk/react/components/organization/organization.module.css
Added new CSS classes for textarea styling with properties: resize: vertical, width: 100%, and max-height: var(--rs-space-17).
Dialog Components
web/sdk/admin/views/organizations/details/layout/invite-users-dialog.tsx, web/sdk/react/views/members/invite-member-dialog.tsx
Applied className props to TextArea components referencing the new CSS classes for styling email input fields.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested labels

bug

Suggested reviewers

  • rohilsurana
  • rohanchkrabrty
  • rsbh
📝 Coding Plan
  • Generate coding plan for human review comments

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

🧹 Nitpick comments (1)
web/sdk/react/components/organization/organization.module.css (1)

39-43: Consider consolidating duplicated textarea-resize rules.

This block is functionally correct, but it mirrors the admin class almost exactly. A shared utility/style token would reduce drift risk if bounds change later.


ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 43d7e66b-0c45-46a3-9dc1-34441965f9d8

📥 Commits

Reviewing files that changed from the base of the PR and between 0755212 and 35f3e3d.

📒 Files selected for processing (4)
  • web/sdk/admin/views/organizations/details/layout/invite-users-dialog.tsx
  • web/sdk/admin/views/organizations/details/layout/layout.module.css
  • web/sdk/react/components/organization/organization.module.css
  • web/sdk/react/views/members/invite-member-dialog.tsx

@coveralls
Copy link

Pull Request Test Coverage Report for Build 23183189752

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 40.56%

Totals Coverage Status
Change from base Build 23181375541: 0.0%
Covered Lines: 14204
Relevant Lines: 35020

💛 - Coveralls

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