Handle svelte-check warnings - Githubissues

techniq / layerchart

Composable Svelte chart components to build a wide range of visualizations

https://www.layerchart.com

MIT License

456 stars 10 forks source link

Handle svelte-check warnings #208

Closed techniq closed 2 weeks ago

techniq commented 3 weeks ago

[x] Handle all svelte-check warnings (A11y, unused variables, etc) so the console is clean
- Possibly leverage  temporarily
[x] Setup CI to run svelte-check and fail build
[ ] If  or // @ts-expect-error are used, properly handle remaining items
[x] Enjoy a peaceful dev experience with improved accessibility

Reference

pnpm check --no-tsconfig --ignore "dist,src/routes
https://www.w3.org/WAI/ARIA/apg/patterns/
- https://www.w3.org/WAI/ARIA/apg/example-index/
https://www.w3.org/wiki/SVG_Accessibility/ARIA_roles_for_charts
https://www.w3.org/TR/graphics-aria-1.0/
https://github.com/techniq/svelte-ux/issues/186

techniq commented 2 weeks ago

Resolved via #207. Type and a11y Improvements will continue, especially during the Svelte 5 migration