Skip to content

fix(SDK): api sdk page layout breaking when key name is too long#1455

Closed
paanSinghCoder wants to merge 3 commits intomainfrom
fix/api-sdk-layout
Closed

fix(SDK): api sdk page layout breaking when key name is too long#1455
paanSinghCoder wants to merge 3 commits intomainfrom
fix/api-sdk-layout

Conversation

@paanSinghCoder
Copy link
Contributor

Summary

  • fix (SDK): Match API page padding with other pages.
  • fix (SDK): content overflowing out of the modal if key name doesn't have space. Added word wrap.

Test Plan

  • Manual testing completed
  • Build and type checking passes

@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 10:14am

@coderabbitai
Copy link

coderabbitai bot commented Mar 17, 2026

📝 Walkthrough

Summary by CodeRabbit

  • Style
    • Enhanced email input text areas in invitation dialogs with improved styling, enabling vertical resizing capability and optimized height constraints for better usability.

Walkthrough

Two email textarea components in invite dialogs receive styling constraints through new CSS classes that restrict vertical resizing and set maximum height limits. The changes are applied identically across admin and React organization member invitation flows.

Changes

Cohort / File(s) Summary
Admin invite users dialog styling
web/sdk/admin/views/organizations/details/layout/invite-users-dialog.tsx, web/sdk/admin/views/organizations/details/layout/layout.module.css
Added className to emails TextArea and defined .invite-users-emails-textarea CSS class with vertical resize constraint and max-height.
React members invite dialog styling
web/sdk/react/views/members/invite-member-dialog.tsx, web/sdk/react/components/organization/organization.module.css
Added className to emails TextArea and defined .invite-member-emails-textarea CSS class with vertical resize constraint and max-height.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 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.

@coveralls
Copy link

coveralls commented Mar 17, 2026

Pull Request Test Coverage Report for Build 23189155038

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.689%

Totals Coverage Status
Change from base Build 23184527020: 0.0%
Covered Lines: 14268
Relevant Lines: 35066

💛 - Coveralls

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/admin/views/organizations/details/layout/layout.module.css (1)

60-64: Consider deduplicating this textarea style across admin/react modules.

This rule is now duplicated with the invite-member textarea class in the React SDK module. A shared utility style (or shared tokenized class) would reduce future style drift.


ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 5dabfaf4-e63d-4a09-a27e-c3a502bb64a8

📥 Commits

Reviewing files that changed from the base of the PR and between d530d58 and 2b8db26.

📒 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

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