Six defects in app/src/components/ui/carousel.tsx, vendored from the shadcn generator and not yet adapted.
:100 — the reInit listener is registered and never removed.:79 — arrow-key handling is capture-phase and calls preventDefault, so it steals Left/Right from any input or textarea inside a slide.:125 — role="region" with aria-roledescription and no accessible name, so a screen reader announces an unnamed region.:140 — the overflow-hidden viewport clips focus rings, and off-screen slides stay in the tab order, so keyboard focus travels to things that cannot be seen.:114 — dead opts.axis fallback: a caller passing axis: "y" is silently overridden.:136 — CarouselContent forwards className to the track rather than the viewport, which is not what a caller would expect.
The keyboard and screen-reader items are the ones worth doing first.
Found during code review of guido/agents-roster-grid and deliberately deferred: real and load-bearing, but a different subject from that branch.
Six defects in
app/src/components/ui/carousel.tsx, vendored from the shadcn generator and not yet adapted.:100— thereInitlistener is registered and never removed.:79— arrow-key handling is capture-phase and callspreventDefault, so it steals Left/Right from any input or textarea inside a slide.:125—role="region"witharia-roledescriptionand no accessible name, so a screen reader announces an unnamed region.:140— theoverflow-hiddenviewport clips focus rings, and off-screen slides stay in the tab order, so keyboard focus travels to things that cannot be seen.:114— deadopts.axisfallback: a caller passingaxis: "y"is silently overridden.:136—CarouselContentforwardsclassNameto the track rather than the viewport, which is not what a caller would expect.The keyboard and screen-reader items are the ones worth doing first.
Found during code review of
guido/agents-roster-gridand deliberately deferred: real and load-bearing, but a different subject from that branch.