fix(button): keep intrinsic labels on one line - #17
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Team Run ID: 📒 Files selected for processing (3)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughButton string labels now render with ChangesButton label behavior
Estimated code review effort: 1 (Trivial) | ~5 minutes Merge Risk: ⚪ Minimal · up to String button labels now remain on one line during intrinsic layout measurement, with coverage for button variants and loading behavior. No current merge-blocking risk remains. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches 💡 1📝 Generate docstrings 💡
🧪 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 |
|
Visit the preview URL for this PR (updated for commit 0e5dee0): https://cpk-ui--pr17-bugfix-solid-button-j30rj1tn.web.app (expires Sun, 13 Sep 2026 03:02:24 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 05244aa8e2a82af676f8a528c78a4bcd59756f99 |
Summary
Verification
No npm publication in this PR. The consumer carries only compiled output while explicit prerelease publication approval is pending.
Summary by CodeRabbit