Handle svelte-check warnings

[techniq]

• Handle all svelte-check warnings (A11y, unused variables, etc) so the console is clean
  • Possibly leverage <!-- svelte-ignore ... --> temporarily
• Setup CI to run svelte-check and fail build
• If <!-- svelte-ignore ... --> or // @ts-expect-error are used, properly handle remaining items
• Rinse/repeat in LayerChart
• 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://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA
• https://www.radix-ui.com/primitives/docs/overview/introduction
• https://react-spectrum.adobe.com/react-aria/Button.html
• https://ariakit.org/
• https://melt-ui.com/docs/introduction

[jycouet]

in user land, if you add in svelte.config.js the function onwarn:

/** @type {import('@sveltejs/kit').Config} */
const config = {
	// ...

	onwarn(warning, defaultHandler) {
		// Do not show 3rd party warnings
		if (warning.filename.includes('node_modules')) return

		defaultHandler(warning)
	},

	// ...
}

export default config

I find it nice until it's "fixed" 😉

[techniq]

Making good progress in PR #409

[techniq]

[jycouet]

That's AMAZING!!!!
Good job

[techniq]

Thanks @jycouet. LayerChart is in progress as well

[techniq]

Sorry for the delay, but Svelte UX 0.71.0 and LayerChart 0.43.0 now provide a much improved Typescript and A11y experience, and a clean console on startup 😁. Typescript and A11y improvements will continue, especially during the Svelte 5 migration. I would appreciate any help on on both of these (searching the codebase for <!-- svelte-ignore ... --> or // @ts-expect-error) as well