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

chore(a11y): ignore svelte@4 a11y warnings #1769

Merged
merged 1 commit into from
Jul 13, 2023
Merged

chore(a11y): ignore svelte@4 a11y warnings #1769

merged 1 commit into from
Jul 13, 2023

Conversation

metonym
Copy link
Collaborator

@metonym metonym commented Jul 13, 2023

There are a couple of new a11y rules in svelte@4 that affect this library.

Note that both warnings are actually true; non-interactive elements like div or label should not have forwarded events.

To avoid breaking changes in this PR, I've ignored these warnings for now.

There's already an issue (#1621) to standardize events/props.

@metonym metonym merged commit 174ed3a into master Jul 13, 2023
2 checks passed
@metonym metonym deleted the svelte-4-a11y branch July 13, 2023 15:47
metonym added a commit that referenced this pull request Jul 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant