Skip to content

Commit 9316af5

Browse files
committed
improvement(docs): drop the repeated per-row play icon from the chapters list
The CirclePlay glyph repeated on every chapter row read as noise — a column of identical icons down the rail. The "On this page" TOC this list mirrors has no per-row icons; the timestamps already signal video chapters and the hover highlight signals they're seekable. Rows are now text + time only.
1 parent 0534b64 commit 9316af5

1 file changed

Lines changed: 2 additions & 4 deletions

File tree

apps/docs/components/ui/video-chapters.tsx

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
'use client'
22

33
import { useEffect, useState } from 'react'
4-
import { CirclePlay } from 'lucide-react'
54
import { cn } from '@/lib/utils'
65

76
/** Parse a chapter timestamp ("M:SS" or "H:MM:SS") into seconds. */
@@ -57,12 +56,11 @@ export function VideoChapters({ title = 'Chapters', chapters, className }: Video
5756
new CustomEvent('academy:seek', { detail: { time: parseTime(chapter.time) } })
5857
)
5958
}}
60-
className='flex w-full cursor-pointer items-start gap-2.5 rounded-lg px-2.5 py-2 text-left text-[var(--text-secondary)] text-sm transition-colors hover:bg-[var(--surface-active)] disabled:cursor-default disabled:hover:bg-transparent'
59+
className='flex w-full cursor-pointer items-baseline gap-3 rounded-lg px-2.5 py-2 text-left text-[var(--text-secondary)] text-sm transition-colors hover:bg-[var(--surface-active)] disabled:cursor-default disabled:hover:bg-transparent'
6160
>
62-
<CirclePlay className='mt-0.5 size-4 shrink-0' />
6361
<span className='min-w-0 flex-1 break-words'>{chapter.title}</span>
6462
{chapter.time && (
65-
<span className='mt-0.5 shrink-0 text-[var(--text-muted)] text-xs tabular-nums'>
63+
<span className='shrink-0 text-[var(--text-muted)] text-xs tabular-nums'>
6664
{chapter.time}
6765
</span>
6866
)}

0 commit comments

Comments
 (0)