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.

Sign up for GitHub

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

Jump to bottom

#a11y - Add an existing account and Sign out should be <button> #1366

Closed
luce-carevic opened this issue Jan 21, 2023 · 1 comment · Fixed by #1382
Closed

#a11y - Add an existing account and Sign out should be <button> #1366

luce-carevic opened this issue Jan 21, 2023 · 1 comment · Fixed by #1382
Labels
c: bug Something isn't working s: pending triage Pending Triage

Comments

@luce-carevic
Copy link

luce-carevic commented Jan 21, 2023 •
edited
Loading

2023-01-21-Language Settings Elk

2023-01-21-Outils de développement - Language Settings Elk - httpselk zonesettingslanguage

These interactive elements are not keyboard accessible and they should be <button>.

See Just use button.
@luce-carevic luce-carevic added c: bug Something isn't working s: pending triage Pending Triage labels Jan 21, 2023
@stackblitz
Copy link

stackblitz bot commented Jan 21, 2023

Solve in StackBlitz Codeflow Start a new pull request in StackBlitz Codeflow.

@luce-carevic luce-carevic changed the title Add an existing account and Sign out should be <button>. #a11y - Add an existing account and Sign out should be <button> Jan 21, 2023
@LittleSound LittleSound mentioned this issue Jan 22, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: bug Something isn't working s: pending triage Pending Triage
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: add an existing account and Sign out should be <button> LittleSound/elk
1 participant
@luce-carevic