Skip to content

Commit 2af9d97

Browse files
committed
fix(workspace): remove file tree label tooltips
1 parent fe44d83 commit 2af9d97

2 files changed

Lines changed: 6 additions & 14 deletions

File tree

packages/web/src/features/workspace/views/shared/file-tree-panel.test.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -940,7 +940,7 @@ describe("FileTreePanel", () => {
940940
expect(await screen.findByRole("dialog")).toBeInTheDocument();
941941
});
942942

943-
it("uses shared tooltips for file-tree action triggers and row path labels", () => {
943+
it("uses shared tooltips for file-tree action triggers without showing row path labels", () => {
944944
const store = createStore();
945945
store.set(wsClientAtom, { sendCommand: vi.fn().mockResolvedValue({}) } as never);
946946
store.set(
@@ -1008,11 +1008,11 @@ describe("FileTreePanel", () => {
10081008

10091009
fireEvent.mouseLeave(deleteFileButton);
10101010
fireEvent.mouseEnter(directoryLabel);
1011-
expect(screen.getByRole("tooltip")).toHaveTextContent("src");
1011+
expect(screen.queryByRole("tooltip")).toBeNull();
10121012

10131013
fireEvent.mouseLeave(directoryLabel);
10141014
fireEvent.mouseEnter(fileLabel);
1015-
expect(screen.getByRole("tooltip")).toHaveTextContent("src/app.tsx");
1015+
expect(screen.queryByRole("tooltip")).toBeNull();
10161016
});
10171017

10181018
it("keeps expanded directories populated after refreshing the file tree", async () => {

packages/web/src/features/workspace/views/shared/file-tree-panel.tsx

Lines changed: 3 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -292,14 +292,8 @@ const FileSearchResultRow: FC<FileSearchResultRowProps> = ({
292292
</span>
293293

294294
<span className="tree-search-labels">
295-
<Tooltip content={node.path}>
296-
<span className="tree-label">{node.name}</span>
297-
</Tooltip>
298-
{dirName ? (
299-
<Tooltip content={node.path}>
300-
<span className="tree-search-path">{dirName}</span>
301-
</Tooltip>
302-
) : null}
295+
<span className="tree-label">{node.name}</span>
296+
{dirName ? <span className="tree-search-path">{dirName}</span> : null}
303297
</span>
304298

305299
<div className="tree-item-actions">
@@ -398,9 +392,7 @@ const FileTreeNode: FC<FileTreeNodeProps> = ({
398392
<Icon size={14} />
399393
</span>
400394

401-
<Tooltip content={node.path}>
402-
<span className="tree-label">{node.name}</span>
403-
</Tooltip>
395+
<span className="tree-label">{node.name}</span>
404396

405397
<div className="tree-item-actions">
406398
{isFolder ? (

0 commit comments

Comments
 (0)