Uploaded image for project: 'Compass '
  1. Compass
  2. COMPASS-5847

Fix/remove compass-crud's disabled eslint accessibility rule breaks

    • 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

            Assignee:
            Unassigned Unassigned
            Reporter:
            rhys.howell@mongodb.com Rhys Howell
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: