fix(webapp): use native checkbox label semantics - #4697
Conversation
|
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
b69c00e to
ed33b99
Compare
ed33b99 to
f436aa1
Compare
ea472f1 to
098f636
Compare
@trigger.dev/build
trigger.dev
@trigger.dev/core
@trigger.dev/python
@trigger.dev/react-hooks
@trigger.dev/redis-worker
@trigger.dev/rsc
@trigger.dev/schema-to-json
@trigger.dev/sdk
commit: |
## Summary Replace mouse-only dashboard actions with native buttons. Copy, remove, and stop-generation controls now expose keyboard focus and accessible names. Hover-revealed actions remain mounted so keyboard users can discover them, and a decorative clipboard icon no longer captures clicks. Base: [#4697](#4697)
Summary
Use native label and checkbox behavior for
CheckboxWithLabeland enforcejsx-a11y/no-noninteractive-element-interactions.The component no longer simulates checkbox activation with click handlers on non-interactive wrappers. Native change events now drive the controlled checked state.
Base: #4696