v2: Explicitly specifying the implicit role of a fieldset element changes the styling about pico HOT 3 OPEN

@willclarktech, thanks.
What tool do you use to see some a11y complains?
I can't see any in Chrome or with Visual Studio linting tools.

I would not remove role="group" now to avoid a breaking change.
But eventually we could change the documentation to use a div instead of a fieldset.
Not sure yet.

from pico.

Hm, I thought it was standard a11y linting, but it seems this is raised by Svelte for VSCode: https://svelte.dev/docs/accessibility-warnings#a11y-no-redundant-roles

To be clear, this isn't a blocker for me. I just thought default roles were worth considering.

from pico.

As role="group" can be used with a div, and we can't ship a breaking change, we should not change anything.
Perhaps the documentation could be improved.

from pico.