Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[a11y]: no-autofocus vuejs-accessibility - address cases where this eslint rule is disabled in arpa_reporter #3894

Open
lsr-explore opened this issue Jan 19, 2025 · 0 comments

Comments

@lsr-explore
Copy link
Contributor

Why is this issue important?

adding autofocus can be disruptive to a screen reader user experience.

vuejs-accessibility reported

no-auto-focus rule documentation

Note: Disabling the rule at the line level did not work to suppress the rule, so it was added at the top of the file.

Current State

Rule is disabled in the code

src/arpa_reporter/views/LoginView.vue

Image

Expected State

Rule should not be disabled in the code

Implementation Plan

  • Run with a screen reader to assess the impact. It may be that in some cases, autofocus is beneficial.
  • File a ticket with the eslint-plugin-vuejs-accessibility github project to be able to disable the rule for just one line.
  • Once the plugin has been updated, move the disable line to the code that is being flagged.

Relevant Code Snippets

n/a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 🆕 New
Development

No branches or pull requests

1 participant