fix: unwanted overflow in recorder

[amhsirak]

Summary by CodeRabbit

Release Notes

• Style
  • Increased output preview area height for improved content visibility
  • Enhanced spacing and margins in recording save interface
  • Optimized layout rendering on recording pages

[coderabbitai]

Caution

Review failed

The pull request is closed.

Walkthrough

Minor styling and layout adjustments across UI components: Paper component positioning, container margins, responsive height calculations, and conditional rendering logic for page layout handling.

Changes

Cohort / File(s)	Summary
RightSidePanel styling src/components/recorder/RightSidePanel.tsx	Added position: relative and border: none to Paper component (#browser-actions) for layout refinement
BrowserRecordingSave styling src/components/browser/BrowserRecordingSave.tsx	Added marginLeft: '10px' to container div and formatted height property with trailing comma
Responsive dimension calculation src/helpers/dimensionUtils.ts	Increased outputPreviewHeight multiplier from 0.7 to 0.9 in getResponsiveDimensions function
PageWrapper conditional rendering src/pages/PageWrapper.tsx	Removed recording page condition from block layout enforcement; now only auth pages trigger block layout via Box component

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

• Changes are mostly styling and minor calculations with no complex logic alterations
• Spread across 4 files but each change is isolated and straightforward
• Primary focus: verify visual consistency of layout changes and confirm the height multiplier adjustment doesn't cause unintended overflow

Possibly related PRs

• feat: enhance screencast quality #491: Modifies getResponsiveDimensions calculations in dimensionUtils.ts with responsive multiplier adjustments
• feat: main layout revamp #865: Adjusts PageWrapper.tsx conditional rendering logic for page layout handling
• Revert "fix: recorder ui bug " #873: Updates RightSidePanel.tsx styling for scroll container and panel layout

Suggested labels

Type: Bug, Scope: UI/UX

Poem

🐰 Whiskers twitching with delight,
Margins nudged and heights just right,
Positions set with gentle care,
Layout polished, clean and fair,
UI blooms in autumn's light!

Pre-merge checks and finishing touches

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'fix: unwanted overflow in recorder' clearly and directly describes the main change - addressing overflow issues in the recorder component by adjusting layout and sizing behavior.

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between c878939 and 5c8da47.

📒 Files selected for processing (2)

• src/components/browser/BrowserRecordingSave.tsx (1 hunks)
• src/pages/PageWrapper.tsx (3 hunks)

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[amhsirak]

@RohitR311 pls confirm if this actually solves the recorder overflow.