Question for Code Reviewer:
Should I merge this into master or into a feature branch? I wasn't sure if we would have additional liquid changes for accessibility work beyond this one.
Also I figured I'd get feedback on changes in Dugway before repeating this work in BC 1.0 and Storefront.
I opted for aria-label instead of aria-current on the current page span.
VO was not picking up aria-current.
Question for Code Reviewer: Should I merge this into master or into a feature branch? I wasn't sure if we would have additional liquid changes for accessibility work beyond this one.
Also I figured I'd get feedback on changes in Dugway before repeating this work in BC 1.0 and Storefront.
I opted for aria-label instead of aria-current on the current page span. VO was not picking up aria-current.
Resources consulted: https://www.a11ymatters.com/pattern/pagination/ https://a11y-style-guide.com/style-guide/section-navigation.html#kssref-navigation-pagination
Fixes https://www.pivotaltracker.com/story/show/169827401 (and related stories for other themes)