Skip to content

Restore right panel inset when maximized (#3555) - #125

Merged
aaditagrawal merged 1 commit into
mainfrom
sync/upstream-20260626-right-panel-inset
Jun 26, 2026
Merged

Restore right panel inset when maximized (#3555)#125
aaditagrawal merged 1 commit into
mainfrom
sync/upstream-20260626-right-panel-inset

Conversation

@aaditagrawal

@aaditagrawal aaditagrawal commented Jun 26, 2026

Copy link
Copy Markdown
Owner

What Changed

Why

UI Changes

Checklist

  • This PR is small and focused
  • I explained what changed and why
  • I included before/after screenshots for any UI changes
  • I included a video for animation/interaction changes

Summary by CodeRabbit

  • Bug Fixes
    • Adjusted the tab bar spacing in the right panel for maximized inline views, improving alignment with the desktop title bar and reducing visual overlap.

@coderabbitai

coderabbitai Bot commented Jun 26, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

Pull request was closed or merged during review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 31311172-515c-4180-8d84-ed3194c4f011

📥 Commits

Reviewing files that changed from the base of the PR and between 70cac9d and d603e50.

📒 Files selected for processing (1)
  • apps/web/src/components/RightPanelTabs.tsx

📝 Walkthrough

Walkthrough

Adds a titlebar inset class to the right panel tab bar when it is in inline mode and maximized.

Changes

Right panel tab bar inset

Layer / File(s) Summary
Inline maximized inset class
apps/web/src/components/RightPanelTabs.tsx
Imports COLLAPSED_SIDEBAR_TITLEBAR_INSET_CLASS and applies it to the tab bar wrapper in inline maximized mode.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 I hopped to the tab bar with care,
And tucked in a snug little inset there.
In inline mode, maximized bright,
The titlebar fit feels just right.
Thump-thump! ✨

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (2 warnings)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description is only the unfilled template and does not explain what changed, why, or show UI evidence. Fill in What Changed, Why, and UI Changes if applicable, and update the checklist with the relevant items.
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 (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely matches the main change: restoring the right panel inset when maximized.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ 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 sync/upstream-20260626-right-panel-inset

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.

@github-actions github-actions Bot added the vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. label Jun 26, 2026

aaditagrawal commented Jun 26, 2026

Copy link
Copy Markdown
Owner Author

Merge activity

  • Jun 26, 3:03 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Jun 26, 3:05 PM UTC: Graphite rebased this pull request as part of a merge.
  • Jun 26, 3:05 PM UTC: @aaditagrawal merged this pull request with Graphite.

@aaditagrawal
aaditagrawal changed the base branch from sync/upstream-20260626-chat-scroll-list to graphite-base/125 June 26, 2026 15:04
@aaditagrawal
aaditagrawal changed the base branch from graphite-base/125 to main June 26, 2026 15:04
@aaditagrawal
aaditagrawal force-pushed the sync/upstream-20260626-right-panel-inset branch from 57be962 to d603e50 Compare June 26, 2026 15:04
@aaditagrawal
aaditagrawal merged commit c64d84a into main Jun 26, 2026
11 of 12 checks passed
@aaditagrawal
aaditagrawal deleted the sync/upstream-20260626-right-panel-inset branch August 1, 2026 19:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS 0-9 effective changed lines (test files excluded in mixed PRs). vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants