Fix Svelte 4 a11y warnings

[AdrianGonz97]

Current Behavior

These are denoted with a TODO: Remove for V2 tag so that they may be easily searchable.

Expected Behavior

No response

Steps To Reproduce

No response

Link to Reproduction / Stackblitz

No response

More Information

No response

[Mahmoud-zino]

Since the AppRailTile has also an issue with a11y, It could be a good opportunity to refactor it and implement keyboard support.

[ollema]

Just wanted to mention that I believe that one of the warnings was not silenced:

7:06:18 PM [vite-plugin-svelte] /node_modules/.pnpm/@skeletonlabs+skeleton@1.9.0_svelte@4.0.3/node_modules/@skeletonlabs/skeleton/dist/components/Ratings/Ratings.svelte:29:3 A11y: <svelte:element> with click handler must have an ARIA role

Maybe this was known?

[Mahmoud-zino]

I would like to work on this issue 🙏.

[endigo9740]

@Mahmoud-zino I might have you help me with one, but I really need to be hands on for anything a11y related. There's a lot of rules and we've gotta make sure changes are implemented in a very specific manner. In this case I know we'll be refactoring a number of components, but I don't know the specifics needed. Each will be determined on a case-by-case basis - even though we might end up making similar changes in multiple places.

I'll try to ping you when I circle back to this though. Thanks!

[endigo9740]

Let's review these specific changes alongside addressing the a11y concerns:

• Modify styling when a Tab is disabled. #1752 (comment)
• Add Disabled stylings to App Rail Tile/Anchor #1890

[endigo9740]

I'm closing this as we will likely not have time to fix this in v2, and v3 will includes rewrites as we port to Svelte 5. Most, if not all, of these issues should be handled along the way. This means this issue is no longer relevant to active tasks.