fix: prevent API key text overflow in create modal#4657
Conversation
Add wordBreak: "break-all" to the API key display span so long keys wrap instead of overflowing outside the modal. Fixes Agenta-AI#4645
|
|
|
Hi @muhamedfazalps, thanks for opening a pull request. 🙏 This PR was automatically closed because it does not yet meet our contribution requirements:
We ask for this so every change is documented and demonstrably tested before review. How to get it reopened See the Contributing guide and Creating your first PR. If you think this was closed in error, leave a comment and a maintainer will take a look. |
|
@muhamedfazalps is attempting to deploy a commit to the agenta projects Team on Vercel. A member of the Team first needs to authorize it. |
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
Disabled knowledge base sources:
📝 WalkthroughSummary by CodeRabbit
WalkthroughThis PR fixes text overflow in the API key display modal by adding a ChangesAPI Key Display Styling
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
Summary
The API key created success modal displays the generated key in a monospace
<span>with no text wrapping. Long API keys (40+ characters in monospace) overflow outside the 650px modal, making them unreadable and partially hiding the copy button.Change
Added
wordBreak: "break-all"to the API key display span's style object inweb/oss/src/components/pages/settings/APIKeys/APIKeys.tsx. This causes long keys to wrap to multiple lines within the modal instead of overflowing.Testing
If this fix helped, consider buying me a coffee! ☕
