This will allow the users to click the buttons while disabling the raw text selection.
Please also consider adding title attributes with descriptions to these buttons.
Thanks a lot for improving the selection of the buttons!
But I also noticed that the gallery above is not linked anymore to the navigation buttons. Was this an intentional change or was a bug introduced?
Also again, some descriptive text would be helpful.
IIRC, the behavior was changed here. In the previous iteration, clicking the button would select and activate the next item in the “image bar” and move the active element into the viewport when necessary. The activation also changed the top image. to the currently selected item in the “image bar”. That allowed us to use the buttons to navigate through the available images.
Currently, the buttons only scroll the “image bar” without activating the items.
In a perfect world:
The buttons would allow the user to navigate through the items. Each click would activate the next or the previous item, which is visualized as an active target with an outline (currently a green tint that is hard to see) for visually impaired users to be (at least WCAG2.0 compliant). Each button should have an aria-label and a title that indicates the action that is triggered by clicking the button. The thumbnails and the active status should be linked to the current state of the carousel. Also when using the alternative swiping on the top image, the active state is currently not reflected (visually), but it at least seems to scroll the currently active “button/thumbnail” into the view.
If I am not mistakenly wrong, u are currently using swiper for the gallery.