mirror of
https://github.com/navidrome/navidrome.git
synced 2026-10-18 07:30:29 +00:00
fix(ui): use a focusable div for the lyrics resizer and hide the decorative lyrics icon
A button with role=separator mixes button activation semantics into the resizer; a tabbable div keeps it a plain focusable separator. The lyrics toolbar icon is decorative since its button already has an aria-label. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016L1sBqsygMkdhf583gTEnE
This commit is contained in:
parent
99e5cdc430
commit
d5fb60bfe2
@ -256,8 +256,8 @@ const LyricsSidebar = ({
|
||||
aria-hidden={!interactive}
|
||||
inert={interactive ? undefined : ''}
|
||||
>
|
||||
<button
|
||||
type="button"
|
||||
<div
|
||||
tabIndex={0}
|
||||
ref={resizerRef}
|
||||
className={classes.resizer}
|
||||
data-testid="lyrics-sidebar-resizer"
|
||||
|
||||
@ -130,6 +130,7 @@ const PlayerToolbar = ({
|
||||
aria-pressed={lyricsActive}
|
||||
>
|
||||
<RiFileMusicLine
|
||||
aria-hidden="true"
|
||||
className={!isDesktop ? classes.mobileIcon : undefined}
|
||||
/>
|
||||
</IconButton>
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user