Fix Svelte a11y warnings

[ilyakamens]

This PR fixes a few Svelte a11y warnings that appear when building a Svelte project that uses Konsta:

A11y: <svg> with click handler must have an ARIA role
A11y: <div> with click handler must have an ARIA role
A11y: <span> with click handler must have an ARIA role
A11y: <svelte:element> with click handler must have an ARIA role
A11y: visible, non-interactive elements with an on:click event must be accompanied by a keyboard event handler. Consider whether an interactive element such as <button type="button"> or <a> might be more appropriate. See https://svelte.dev/docs/accessibility-warnings#a11y-click-events-have-key-events for more details.

Screenshot of some of the warnings:

Some of these have (had) issues opened for them:

• A11y: visible, non-interactive elements with an on:click event must be accompanied by an on:keydown, on:keyup, or on:keypress event. #92
• A11y: improve a11y capability, adding ARIA roles #155

In the case of #155, it was closed with 59216a6, but that change didn't actually fix the file that the issue referenced (DeleteIcon.svelte).

For "A11y: visible, non-interactive elements with an on:click event must be accompanied by a keyboard event handler," I added a comment to suppress the warning because I wasn't sure what the desired fix was. I'd be happy to revisit that with more guidance if desired.

Versions:

@sveltejs/vite-plugin-svelte at 3.1.1.
konsta at 3.1.3.

Thanks for your consideration.

[nolimits4web]

t0ggles-create konsta

[t0ggles]

Task nolimits4web/KONSTA-90 was created

[t0ggles]

Task nolimits4web/KONSTA-90 status changed to Done

[nolimits4web]

Merged, thanks! I removed role button from irrelevant elements, as they still don't have much sense without key events handler