forked from kenwheeler/slick
-
Notifications
You must be signed in to change notification settings - Fork 48
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support customized text for all aria-labels #86
Comments
The opened PR #88 makes the following changes:
Item / Slide:
|
recrit
pushed a commit
to recrit/accessible-slick-issue-86-customizable-aria-labels
that referenced
this issue
Feb 16, 2023
…d items, slick.min.js is not updated.
@recrit : when will it be released to fix the problem |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The following represents the aria-labels used as of master 61b5322:
aria-label
can be customized with the optionregionLabel
; however, if you do not set aregionLabel
then a hard coded "carousel" is used.aria-label
cannot customized. Thearia-label
is always set to "slide [number of slide]", example "slide 2".Issues
aria-label
oraria-labelledby
that provides more details or is translated by a backend CMS (example Drupal).Desired Changes to support customizing the aria-labels
aria-label
oraria-labelledby
then the slick library should not override it with the hard coded "carousel" or theoptions.regionLabel
.options.customItemLabel
function similar tooptions.customPaging
that is passed theslider
and the slide index. This would allow the caller to pass in a custom label function or use the default that would return "slide [number of slide]".aria-label
oraria-labelledby
then the slick library should not override it with theoptions.customItemLabel
.Generic Example:
Example with Drupal:
The text was updated successfully, but these errors were encountered: