-
Type: Task
-
Resolution: Won't Do
-
Priority: Major - P3
-
Affects Version/s: None
-
Component/s: Accessibility, CRUD, Tech debt
-
None
-
3
-
Not Needed
-
Iteration Dolphin, Iteration Eel, Iteration Fish
In updating `compass-crud` to the shared eslint config of Compass we encountered a number of places where the plugin was using non-interactive html elements as buttons with `onClick` handlers. Mostly in the documents table view. This ticket involves removing the places where we disabled eslint rules so that the lint `check` passes and we have less accessibility issues in Compass.
I tagged most of the places with this ticket number so you can search for that in the code base as well as the eslint-disabled rules below vvv
Rules:
- Visible, non-interactive elements with click handlers must have at least one keyboard listener jsx-a11y/click-events-have-key-events
- Static HTML elements with event handlers require a role jsx-a11y/no-static-element-interactions
- Non-interactive elements should not be assigned mouse or keyboard event listeners jsx-a11y/no-noninteractive-element-interactions
- depends on
-
COMPASS-5824 Update `compass-crud` package config to shared Compass config
- Closed
- is related to
-
COMPASS-6139 Refactor usage of UNSAFE_ methods in insert-document-dialog
- Closed