Replace button tag with div

[monolithed]

I've come across similar tickets with semantic markup, but it seems that the 'button' should not be a tag because it may contain block-level elements, which violates semantic markup principles. It would be great to replace it with a

.

[nerdyman]

Thanks for raising this @monolithed. I've looked at other slider role examples and they do indeed use div tags, not button.

I'll look at resolving this before releasing v4. Also open to PRs!