Skip to content

Calls admin link layout#708

Merged
kentcdodds merged 1 commit intomainfrom
cursor/calls-admin-link-layout-f1d4
Feb 25, 2026
Merged

Calls admin link layout#708
kentcdodds merged 1 commit intomainfrom
cursor/calls-admin-link-layout-f1d4

Conversation

@kentcdodds
Copy link
Owner

@kentcdodds kentcdodds commented Feb 25, 2026

Improve calls admin page layout by wrapping long links in notes and widening the detail pane.


Open in Web Open in Cursor 


Note

Low Risk
Purely presentational Tailwind class changes with no data, routing, or security logic impact; main risk is minor layout regressions at responsive breakpoints.

Overview
Improves the Calls Admin UI layout for readability and space usage.

On the call detail page, notes now force-wrap long unbroken text/URLs via added Tailwind classes (break-words and overflow-wrap:anywhere) to prevent horizontal overflow.

In the Calls Admin layout grid, the sidebar is narrowed and the main detail pane is widened on large screens by adjusting lg:col-span/lg:col-start values.

Written by Cursor Bugbot for commit 9a4cc7b. This will update automatically on new commits. Configure here.

Summary by CodeRabbit

  • Style
    • Improved text wrapping behavior for call notes to better handle long text without overflow
    • Adjusted responsive grid layout proportions between sidebar and main content area for enhanced visual balance

Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
@cursor
Copy link

cursor bot commented Feb 25, 2026

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@coderabbitai
Copy link

coderabbitai bot commented Feb 25, 2026

📝 Walkthrough

Walkthrough

Two admin call interface files receive CSS styling adjustments: text wrapping improvements via additional classes for call notes, and responsive grid column rebalancing to shift layout proportions at large breakpoints.

Changes

Cohort / File(s) Summary
Admin Calls Styling
app/routes/calls_.admin/$callId.tsx, app/routes/calls_.admin/_layout.tsx
Enhanced text wrapping in notes section with break-words and [overflow-wrap:anywhere] classes; rebalanced responsive grid layout by adjusting sidebar from lg:col-span-4 to lg:col-span-3 and content area from lg:col-span-7 lg:col-start-6 to lg:col-span-8 lg:col-start-5.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • PR #580: Modifies the same admin call route files with CSS class and responsive grid span adjustments for the calls admin layout.

Poem

🐰 A hop and a skip through CSS lanes,
Where columns dance and text flows free,
Break-words flutter, wrapping's clean,
Grid rebalances gracefully,
Pixels pirouette—such responsive delight!

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Calls admin link layout' directly relates to the main changes: improving text wrapping for links/notes in the calls admin page and rebalancing the layout grid.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch cursor/calls-admin-link-layout-f1d4

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.

@kentcdodds kentcdodds marked this pull request as ready for review February 25, 2026 19:17
@kentcdodds kentcdodds merged commit bd91621 into main Feb 25, 2026
8 checks passed
@kentcdodds kentcdodds deleted the cursor/calls-admin-link-layout-f1d4 branch February 25, 2026 19:57
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