Skip to content

fix: Firefox session layout was clipping content#1061

Merged
Gkrumbach07 merged 1 commit intomainfrom
fix-layout-firefox
Mar 26, 2026
Merged

fix: Firefox session layout was clipping content#1061
Gkrumbach07 merged 1 commit intomainfrom
fix-layout-firefox

Conversation

@dgutride
Copy link
Copy Markdown
Contributor

Fixes firefox layout issues - and an issue I Noticed when filling the entire screen with content where it slid under the top header:

Screenshot 2026-03-26 at 2 36 25 PM

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 26, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: a5b5a8c3-ef14-4111-a387-99b45d3909b7

📥 Commits

Reviewing files that changed from the base of the PR and between ee88acc and e14f15d.

📒 Files selected for processing (1)
  • components/frontend/src/app/projects/[name]/sessions/[sessionName]/components/new-session-view.tsx

Walkthrough

Added auto-resize functionality to a textarea component with a useEffect hook that dynamically adjusts height based on scroll height. Updated container and textarea CSS classes to support the new resize behavior when prompt content changes.

Changes

Cohort / File(s) Summary
Textarea Auto-Resize
components/frontend/src/app/projects/[name]/sessions/[sessionName]/components/new-session-view.tsx
Added useEffect hook to auto-resize prompt textarea by updating height from scrollHeight. Changed container class from h-full to min-h-full and textarea from overflow-y-auto to overflow-y-hidden to support dynamic sizing.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

🚥 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 clearly and specifically describes the main change: fixing a Firefox session layout clipping issue.
Description check ✅ Passed The description is directly related to the changeset, explaining the Firefox layout problems and content clipping issues being fixed.

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

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix-layout-firefox

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.

@dgutride
Copy link
Copy Markdown
Contributor Author

Fixes #1024

@Gkrumbach07 Gkrumbach07 merged commit ea31419 into main Mar 26, 2026
34 checks passed
@Gkrumbach07 Gkrumbach07 deleted the fix-layout-firefox branch March 26, 2026 18:56
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