-
Notifications
You must be signed in to change notification settings - Fork 16
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
Add option to render the carousel track as an unordered list #119
Conversation
|
@@ -144,7 +147,7 @@ export default | |||
|
|||
# Render the track and slotted slides | |||
render: (create) -> | |||
create 'div', | |||
create @trackHTMLElement, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We could possibly make this not require a new prop by checking if the firs slide returned by makeSlides()
has an element of li
. Can you console.log it and see if has a type
or element
property or something like that so we can try and auto-determine this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just updated the pr with the requested change
Little syntax tweaks
@lemoswilson I tagged this as |
This is a change requested as part of the HFO ADA audit made by Level Access.
On the page https://www.happyfamilyorganics.com/our-story/ we have a button carousel that is used to navigate between the tabs of a block, and Level Access wrote the following about it:
_The below-mentioned textual links below "Take a Journey Through Our Story" heading text visually appear as list items but are not marked using HTML list markup. When links are properly grouped, it facilitates labeling the group, skipping past the group, and helps to divide large blocks of information into more manageable components.
Following instances are affected:
[User Impact]
Screen reader users will benefit if the content has been structured using list markup._
This small change should provide a better user experience for screen reader users.