[saddlepaddle]

Summary

The v2 workspace sidebar's tab header uses the inverted tab style where the active tab draws a 1px border on all sides. When the leftmost tab (Files) is active, its left border sits flush against the ResizablePanel's border-l, rendering a 2px doubled divider across the tab-header row.

Fix: make the first tab's left border transparent — the resizable panel already draws the sidebar's left edge. Inactive tabs already had a transparent left border, so nothing else changes.

Test Plan

• Open a v2 workspace with the right sidebar visible and the Files tab active — the divider between the panes and the sidebar is a single 1px line across the tab-header row
• Switch between Files/Changes/Review tabs — no layout shift, divider stays single

https://claude.ai/code/session_015mBAauusSAewcv7JHm5pAw

---

Summary by cubic

Removes the doubled 2px divider between the main pane and the right sidebar by making the first sidebar tab’s left border transparent. Ensures a single, consistent 1px divider across the tab header with no layout shift when switching tabs.

^{Written for commit 7ac34b1. Summary will update on new commits.}

Summary by CodeRabbit

• Style
  • Improved the workspace sidebar’s visual alignment by preventing a duplicated border on the first tab next to the resizable panel.

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 2507cfb6-68fd-44e1-9795-10fadbd08a93

📥 Commits

Reviewing files that changed from the base of the PR and between 68672e4 and 7ac34b1.

📒 Files selected for processing (1)

• apps/desktop/src/renderer/routes/_authenticated/_dashboard/v2-workspace/$workspaceId/components/WorkspaceSidebar/components/SidebarHeader/SidebarHeader.tsx

---

📝 Walkthrough

Walkthrough

The sidebar tab mapping now tracks tab indexes. The first tab receives a transparent left border to avoid duplicating the resizable panel’s sidebar edge.

Changes

Sidebar tab border adjustment

Layer / File(s)	Summary
Apply first-tab border styling apps/desktop/src/renderer/routes/_authenticated/_dashboard/v2-workspace/$workspaceId/components/WorkspaceSidebar/components/SidebarHeader/SidebarHeader.tsx	The tab iteration captures each tab index. The first tab conditionally receives border-l-transparent; later tabs retain the existing classes.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

• superset-sh/superset#5703: Both changes adjust SidebarHeader.tsx tab layout and border styling.
• superset-sh/superset#5824: Both changes modify tab styling and sidebar border behavior in SidebarHeader.tsx.
• superset-sh/superset#5851: Both changes modify tab mapping and border styling in SidebarHeader.tsx.

Suggested reviewers: avipeltz, kitenite

✨ 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-doubled-borders

---

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.}

[github-actions]

🧹 Preview Cleanup Complete

The following preview resources have been cleaned up:

• ⚠️ Neon database branch

Thank you for your contribution! 🎉

[greptile-apps]

Greptile Summary

The PR removes the doubled divider beside the v2 workspace sidebar by making the first tab’s left border transparent while retaining the resizable panel’s border.

• Passes the tab index through the sidebar-header render loop.
• Applies border-l-transparent only to the leftmost tab.
• Preserves border width, avoiding layout shifts when switching tabs.

Confidence Score: 5/5

The PR appears safe to merge with no actionable defects identified.

The change is narrowly scoped to the first rendered tab’s left border color and preserves the existing border width and fixed tab ordering.

Important Files Changed

Filename	Overview
apps/desktop/src/renderer/routes/_authenticated/_dashboard/v2-workspace/$workspaceId/components/WorkspaceSidebar/components/SidebarHeader/SidebarHeader.tsx	Correctly suppresses the first tab’s redundant border color without changing tab dimensions or behavior.

_{Reviews (1): Last reviewed commit: "fix(desktop): drop first sidebar tab's l..." | Re-trigger Greptile}