-
Notifications
You must be signed in to change notification settings - Fork 3
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
How do I style the accordion to have a space between each accordion panel? #3
Comments
@benedikt-buchert Sounds fine to me. |
#4 |
@benedikt-buchert The current markup with dt and dl looks outdated and having some issues with screen readers I guess. The new markup seems to be having buttons and divs. Will update the markup and give provisions for spacing. Is it ok? |
Sounds good the article linked above by Sāra Soueidān gives some great options: https://www.sarasoueidan.com/blog/accordion-markup/ |
I want space each accordion panel.
For that I would need to wrap each
<d>
and<dt>
inside a div.Had a little read on the possible accessibility problems that this might cause: https://cdpn.io/stevef/debug/GxwaoP And it looks like this is fine.
What do you think?
Some further info:
https://www.sarasoueidan.com/blog/accordion-markup/ https://www.scottohara.me/blog/2017/10/25/accordion-release.html
https://inclusive-components.design/tabbed-interfaces/
The text was updated successfully, but these errors were encountered: