Skip to content

Commit 3102607

Browse files
authored
fix: cleanup left sidebar icon placement (#3120)
1 parent 652562f commit 3102607

3 files changed

Lines changed: 8 additions & 6 deletions

File tree

src/browser/components/AgentListItem/AgentListItem.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -118,7 +118,7 @@ export interface DraftAgentListItemProps extends AgentListItemBaseProps {
118118
* before react-dnd has locked into a drag session.
119119
*/
120120
const LIST_ITEM_BASE_CLASSES =
121-
"bg-surface-primary relative flex items-start gap-1.5 rounded-l-sm py-2 pr-2 select-none transition-all duration-150";
121+
"bg-surface-primary relative flex items-start gap-1.5 rounded-l-sm py-2 pr-1.5 select-none transition-all duration-150";
122122

123123
const HIDE_INLINE_ACTIONS_ON_MOBILE_TOUCH =
124124
"[@media(max-width:768px)_and_(hover:none)_and_(pointer:coarse)]:invisible [@media(max-width:768px)_and_(hover:none)_and_(pointer:coarse)]:pointer-events-none";

src/browser/components/ProjectSidebar/ProjectSidebar.tsx

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -284,8 +284,9 @@ function useWorkspaceAttentionSubscription(
284284
// Keep the project header visible while scrolling through long workspace lists.
285285
// Project rows are also drag handles, so disable text selection to avoid
286286
// highlighting the whole sidebar before a reorder gesture locks in.
287+
// pr-2 matches AgentListItem LIST_ITEM_BASE_CLASSES so project kebab aligns with workspace rows.
287288
const PROJECT_ITEM_BASE_CLASS =
288-
"group sticky top-0 z-10 py-2 pl-2 pr-3 flex select-none items-center border-l-transparent bg-surface-primary transition-colors duration-150";
289+
"group sticky top-0 z-10 py-2 pl-2 pr-1 flex select-none items-center border-l-transparent bg-surface-primary transition-colors duration-150";
289290

290291
function getProjectFallbackLabel(projectPath: string): string {
291292
const abbreviatedPath = PlatformPaths.abbreviate(projectPath);
@@ -2046,7 +2047,7 @@ const ProjectSidebarInner: React.FC<ProjectSidebarProps> = ({
20462047
}}
20472048
aria-label={`New chat in ${projectName}`}
20482049
data-project-path={projectPath}
2049-
className="text-content-secondary hover:bg-hover hover:border-border-light mr-1 flex h-5 w-5 shrink-0 cursor-pointer items-center justify-center rounded border border-transparent bg-transparent text-sm leading-none transition-all duration-200"
2050+
className="text-content-secondary hover:bg-hover hover:border-border-light flex h-5 w-5 shrink-0 cursor-pointer items-center justify-center rounded border border-transparent bg-transparent text-sm leading-none transition-all duration-200"
20502051
>
20512052
<Plus className="h-4 w-4 shrink-0" strokeWidth={1.8} />
20522053
</button>

src/browser/components/SectionHeader/SectionHeader.tsx

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ import {
88
Palette,
99
Pencil,
1010
Trash2,
11+
Plus,
1112
} from "lucide-react";
1213
import type { SectionConfig } from "@/common/types/project";
1314
import { Tooltip, TooltipTrigger, TooltipContent, TooltipIfPresent } from "../Tooltip/Tooltip";
@@ -106,7 +107,7 @@ export const SectionHeader: React.FC<SectionHeaderProps> = ({
106107

107108
return (
108109
<div
109-
className="group relative ml-4 flex items-center gap-1 py-1.5 pr-2 pl-3 select-none"
110+
className="group relative ml-4 flex items-center gap-1 py-1.5 pr-1 pl-3 select-none"
110111
data-section-id={section.id}
111112
>
112113
{/* Expand/Collapse Button */}
@@ -180,7 +181,7 @@ export const SectionHeader: React.FC<SectionHeaderProps> = ({
180181
)}
181182

182183
{/* Right-side controls: add chat + section actions */}
183-
<div className="flex items-center gap-0.5 opacity-0 transition-opacity group-hover:opacity-100 [@media(hover:none)_and_(pointer:coarse)]:opacity-100">
184+
<div className="flex items-center opacity-0 transition-opacity group-hover:opacity-100 [@media(hover:none)_and_(pointer:coarse)]:opacity-100">
184185
{/* Add Chat — always visible on touch devices */}
185186
<Tooltip>
186187
<TooltipTrigger asChild>
@@ -189,7 +190,7 @@ export const SectionHeader: React.FC<SectionHeaderProps> = ({
189190
className="text-secondary hover:text-foreground hover:bg-hover flex h-5 w-5 cursor-pointer items-center justify-center rounded border-none bg-transparent p-0 text-sm transition-colors"
190191
aria-label="New chat in section"
191192
>
192-
+
193+
<Plus className="h-4 w-4" />
193194
</button>
194195
</TooltipTrigger>
195196
<TooltipContent>New chat</TooltipContent>

0 commit comments

Comments
 (0)