Restore right panel inset when maximized#3555
Merged
Merged
Conversation
- Apply the collapsed sidebar titlebar inset in maximized desktop mode - Keep the right panel tab bar aligned with native window controls
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
github-actions
Bot
added
vouch:trusted
![macroscopeapp[bot]](https://avatars.githubusercontent.com/in/900172?s=60&v=4){kind=small}
Contributor
ApprovabilityVerdict: Approved Minor UI styling fix that conditionally applies a padding class when the right panel is maximized. The change is purely visual with no runtime behavior impact. You can customize Macroscope's approvability policy. Learn more. |
- Avoid applying the collapsed sidebar titlebar inset in non-inline layouts - Preserve desktop titlebar spacing while keeping inline maximized tabs aligned
macroscopeapp
Bot
dismissed
their stale review
June 25, 2026 17:03
Dismissing prior approval to re-evaluate 2c76fcc
juliusmarminge
enabled auto-merge (squash)
June 25, 2026 17:05
aaditagrawal
pushed a commit
to aaditagrawal/t3code
that referenced
this pull request
Jun 26, 2026
aaditagrawal
added a commit
to aaditagrawal/t3code
that referenced
this pull request
Jun 26, 2026
<!--⚠️ READ BEFORE OPENING⚠️ We are not actively accepting contributions right now. You can still open a PR, but please do so knowing there is a high chance we may close it without merging it, or never review it. - Small, focused PRs are strongly preferred. Bug fixes are most likely to be merged. - New features will most likely just annoy us. - 1,000+ line PRs with a bunch of new features will probably get you banned from the repo. --> ## What Changed <!-- Describe the change clearly and keep scope tight. --> ## Why <!-- Explain the problem being solved and why this approach is the right one. --> ## UI Changes <!-- If this PR changes UI, include clear before/after screenshots. If the change involves motion or interaction, include a short video. Delete this section if not applicable. --> ## 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
chanyeinthaw
pushed a commit
to sats-lab/pulse
that referenced
this pull request
Jun 28, 2026
simonbetton
pushed a commit
to simonbetton/reviewer
that referenced
this pull request
Jun 29, 2026
2 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Testing
Note
Low Risk
Single conditional CSS class on the right panel top bar; no auth, data, or API changes.
Overview
Restores correct left padding on the right panel tab bar when the inline panel is maximized in the desktop app, so tabs no longer sit under the collapsed-sidebar / native titlebar region.
RightPanelTabsnow applies the sharedCOLLAPSED_SIDEBAR_TITLEBAR_INSET_CLASS(already used on the main chat and empty-thread headers) only whenmode === "inline"andmaximizedis true.Reviewed by Cursor Bugbot for commit 2c76fcc. Bugbot is set up for automated code reviews on this repo. Configure here.
Note
Restore right panel inset styling when maximized in inline mode
In RightPanelTabs.tsx, the topbar div now conditionally applies
COLLAPSED_SIDEBAR_TITLEBAR_INSET_CLASSwhen the panel is in inline mode and maximized, restoring the correct inset appearance under those conditions.Macroscope summarized 2c76fcc.