Megabit / Blazorise

Blazorise is a component library built on top of Blazor with support for CSS frameworks like Bootstrap, Tailwind, Bulma, AntDesign, and Material.
https://blazorise.com/
Other
3.27k stars 530 forks source link

[Bug]: Stepper is not accessible #5510

Open panmona opened 4 months ago

panmona commented 4 months ago

Blazorise Version

1.5.x

What Blazorise provider are you running on?

None

Link to minimal reproduction or a simple code snippet

https://blazorise.com/docs/components/step

Steps to reproduce

What is expected?

What is actually happening?

What browsers do you see the problem on?

Chrome

Any additional comments?

The problem is that the "buttons" are clickable divs, which should never be used. (for reference e.g: https://www.a11yproject.com/checklist/#controls and https://benmyers.dev/blog/clickable-divs/#or) The stepper buttons should rather use the HTML button element with CSS overrides. In this case we'll get the things that aren't working for free ๐Ÿ†“๐Ÿ˜„

Additionally by using buttons, we would get the expected mouse pointer style for free.

stsrki commented 4 months ago

Thank you for reporting this issue. We already plan to add many accessibility improvements as part of #3705 and #3454, possibly in 1.7 release.

In case you find any other issues, don't hesitate to open a new ticket, as any extra information would greatly help us to figure out what needs to be done.

panmona commented 4 months ago

That's very good to hear that you're committed to improving this ๐Ÿ‘ In that case if I find anything else I'll open another ticket ๐Ÿ‘