Skip to content

Commit 85aa591

Browse files
committed
fix
1 parent b928005 commit 85aa591

3 files changed

Lines changed: 15 additions & 15 deletions

File tree

app/frontend/components/design-system/sections/elements/DropdownMenuSection.tsx

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ import {
99
} from "@/components/ui/dropdown-menu";
1010
import {
1111
ChevronDown,
12-
CircleEllipsis,
12+
Ellipsis,
1313
LogOut,
1414
Settings,
1515
Trash2,
@@ -24,7 +24,7 @@ const code = `import {
2424
DropdownMenuSeparator,
2525
DropdownMenuTrigger,
2626
} from "@/components/ui/dropdown-menu";
27-
import { ChevronDown, CircleEllipsis, LogOut, Settings, Trash2, User, Users } from "lucide-react";
27+
import { ChevronDown, Ellipsis, LogOut, Settings, Trash2, User, Users } from "lucide-react";
2828
2929
{/* Button trigger */}
3030
<DropdownMenu>
@@ -44,7 +44,7 @@ import { ChevronDown, CircleEllipsis, LogOut, Settings, Trash2, User, Users } fr
4444
4545
{/* Item settings dropdown — the standard pattern for row-level
4646
"edit / archive / delete" overflow menus. Bare button (no border or
47-
background), CircleEllipsis icon at strokeWidth 1.5, ink-muted with
47+
background), Ellipsis icon at strokeWidth 1.5, ink-muted with
4848
hover:ink-body. */}
4949
<DropdownMenu>
5050
<DropdownMenuTrigger asChild>
@@ -53,7 +53,7 @@ import { ChevronDown, CircleEllipsis, LogOut, Settings, Trash2, User, Users } fr
5353
aria-label="Settings"
5454
className="inline-flex cursor-pointer items-center justify-center text-ink-muted transition-colors hover:text-ink-body focus:outline-none focus-visible:text-ink-body"
5555
>
56-
<CircleEllipsis className="h-4 w-4" strokeWidth={1.5} />
56+
<Ellipsis className="h-4 w-4" strokeWidth={1.5} />
5757
</button>
5858
</DropdownMenuTrigger>
5959
<DropdownMenuContent align="end">
@@ -126,7 +126,7 @@ export function DropdownMenuSection() {
126126
aria-label="Settings"
127127
className="inline-flex cursor-pointer items-center justify-center text-ink-muted transition-colors hover:text-ink-body focus:outline-none focus-visible:text-ink-body"
128128
>
129-
<CircleEllipsis className="h-4 w-4" strokeWidth={1.5} />
129+
<Ellipsis className="h-4 w-4" strokeWidth={1.5} />
130130
</button>
131131
</DropdownMenuTrigger>
132132
<DropdownMenuContent align="end">
@@ -175,7 +175,7 @@ export function DropdownMenuSection() {
175175
<strong>Item settings dropdown — standard pattern.</strong> For
176176
row-level "edit / archive / delete" overflow menus on listings or
177177
single-item views, always use this exact trigger:{" "}
178-
<code>CircleEllipsis</code> icon, <code>strokeWidth={1.5}</code>,
178+
<code>Ellipsis</code> icon, <code>strokeWidth={1.5}</code>,
179179
no border or background, <code>text-ink-muted</code> with{" "}
180180
<code>hover:text-ink-body</code>. Use <code>h-4 w-4</code> on
181181
listing rows; bump to <code>h-6 w-6</code> when used in a page

app/frontend/components/design-system/sections/elements/ListingsSection.tsx

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ import {
66
DropdownMenuItem,
77
DropdownMenuTrigger,
88
} from "@/components/ui/dropdown-menu";
9-
import { ChevronRight, CircleEllipsis, Folder } from "lucide-react";
9+
import { ChevronRight, Ellipsis, Folder } from "lucide-react";
1010

1111
const code = `<ul className="divide-y divide-hairline overflow-hidden rounded-md border border-hairline bg-page">
1212
{items.map((item) => (
@@ -28,7 +28,7 @@ const code = `<ul className="divide-y divide-hairline overflow-hidden rounded-md
2828
aria-label="Settings"
2929
className="inline-flex cursor-pointer items-center justify-center text-ink-muted transition-colors hover:text-ink-body focus:outline-none focus-visible:text-ink-body"
3030
>
31-
<CircleEllipsis className="h-4 w-4" strokeWidth={1.5} />
31+
<Ellipsis className="h-4 w-4" strokeWidth={1.5} />
3232
</button>
3333
</DropdownMenuTrigger>
3434
<DropdownMenuContent align="end">
@@ -57,7 +57,7 @@ function DemoSettingsMenu() {
5757
aria-label="Settings"
5858
className="inline-flex cursor-pointer items-center justify-center text-ink-muted transition-colors hover:text-ink-body focus:outline-none focus-visible:text-ink-body"
5959
>
60-
<CircleEllipsis className="h-4 w-4" strokeWidth={1.5} />
60+
<Ellipsis className="h-4 w-4" strokeWidth={1.5} />
6161
</button>
6262
</DropdownMenuTrigger>
6363
<DropdownMenuContent align="end">
@@ -129,7 +129,7 @@ export function ListingsSection() {
129129
items in a collection (the typical case), include our standard
130130
settings dropdown at the right edge of each row — see{" "}
131131
<a href="#dropdown-menu">Dropdown menu</a> for the trigger pattern
132-
(bare <code>CircleEllipsis</code>, no border).
132+
(bare <code>Ellipsis</code>, no border).
133133
</li>
134134
</ul>
135135
}

app/frontend/components/design-system/sections/structure/PageHeadersSection.tsx

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ import {
77
DropdownMenuItem,
88
DropdownMenuTrigger,
99
} from "@/components/ui/dropdown-menu";
10-
import { Bell, CircleEllipsis, Search } from "lucide-react";
10+
import { Bell, Ellipsis, Search } from "lucide-react";
1111

1212
const code = `{/* Basic — title row only */}
1313
<div className="border-b border-hairline pb-6">
@@ -29,7 +29,7 @@ const code = `{/* Basic — title row only */}
2929
</div>
3030
3131
{/* Single-item view — page title with the standard item settings dropdown
32-
in the right slot. The CircleEllipsis trigger uses h-6 w-6 here (larger
32+
in the right slot. The Ellipsis trigger uses h-6 w-6 here (larger
3333
than the h-4 w-4 used in listing rows). */}
3434
<div className="border-b border-hairline pb-6">
3535
<div className="flex flex-col gap-4 sm:flex-row sm:items-start sm:justify-between">
@@ -44,7 +44,7 @@ const code = `{/* Basic — title row only */}
4444
aria-label="Settings"
4545
className="inline-flex cursor-pointer items-center justify-center text-ink-muted transition-colors hover:text-ink-body focus:outline-none focus-visible:text-ink-body"
4646
>
47-
<CircleEllipsis className="h-6 w-6" strokeWidth={1.5} />
47+
<Ellipsis className="h-6 w-6" strokeWidth={1.5} />
4848
</button>
4949
</DropdownMenuTrigger>
5050
<DropdownMenuContent align="end">
@@ -173,7 +173,7 @@ export function PageHeadersSection() {
173173
aria-label="Settings"
174174
className="inline-flex cursor-pointer items-center justify-center text-ink-muted transition-colors hover:text-ink-body focus:outline-none focus-visible:text-ink-body"
175175
>
176-
<CircleEllipsis
176+
<Ellipsis
177177
className="h-6 w-6"
178178
strokeWidth={1.5}
179179
/>
@@ -246,7 +246,7 @@ export function PageHeadersSection() {
246246
<strong>Single-item view</strong>: when viewing one item from a
247247
collection, the right slot should hold our standard settings
248248
dropdown — see <a href="#dropdown-menu">Dropdown menu</a>. Use{" "}
249-
<code>h-6 w-6</code> on the <code>CircleEllipsis</code> here
249+
<code>h-6 w-6</code> on the <code>Ellipsis</code> here
250250
(one size larger than in listing rows) to match the page
251251
header's heavier visual weight.
252252
</li>

0 commit comments

Comments
 (0)