Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Add axe to create accessibility reports #823

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

Closed
AlbyIanna wants to merge 1 commit into main from axe
Closed

Add axe to create accessibility reports #823

AlbyIanna wants to merge 1 commit into main from axe

Conversation

@AlbyIanna
Copy link
Contributor

@AlbyIanna AlbyIanna commented Feb 9, 2022

Why

We want to make the Arduino IDE more accessible.

How

We decided to choose axe, a tool that does automated accessibility tests at runtime.

Test instructions

  • download the build and launch the executable
  • open the developer tools (in the menu Help > Toggle Developer Tools)
  • open the Console tab
  • launch the command
runAxe()
  • you will see some coloured lines spawn in the console reporting accessibility issues, like this:
    image
  • once you've run this command and until you close the window, as you interact with the application you will keep getting new reports

@AlbyIanna AlbyIanna changed the title (削除) add axe to create accessibility reports (削除ここまで) (追記) Add axe to create accessibility reports (追記ここまで) Feb 9, 2022
"which": "^1.3.1"
},
"devDependencies": {
"@axe-core/react": "^4.3.2",

This comment was marked as off-topic.

Copy link
Contributor Author

@AlbyIanna AlbyIanna Feb 9, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the comment @kittaakos. Basically we're doing this to help the designers testing the app. We'll definitely change it as dev dependency as soon as we find a better solution.

kittaakos reacted with heart emoji
@per1234 per1234 added topic: accessibility Enabling the use of the software by everyone topic: infrastructure Related to project infrastructure type: enhancement Proposed improvement labels Feb 11, 2022
Copy link
Contributor Author

Closing this because it was open for testing purposes.

@per1234 per1234 added the conclusion: invalid Issue/PR not valid label Jun 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@fstasi fstasi Awaiting requested review from fstasi

@91volt 91volt Awaiting requested review from 91volt

1 more reviewer

@kittaakos kittaakos kittaakos left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

conclusion: invalid Issue/PR not valid topic: accessibility Enabling the use of the software by everyone topic: infrastructure Related to project infrastructure type: enhancement Proposed improvement

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /