Skip to content
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

Tabs refactored not reflected in all documentation #107

Closed
FabienTaillon opened this issue Nov 20, 2015 · 6 comments
Closed

Tabs refactored not reflected in all documentation #107

FabienTaillon opened this issue Nov 20, 2015 · 6 comments
Assignees
Labels

Comments

@FabienTaillon
Copy link

Tabs has been refactored, however the documentation still refers to .slds-tabs__item or .slds-tabs__content, like in the Javacript needs or Component Overview (last 3 lines) sections.

Also, it's said that "Each tab panel has an aria-labelledby attribute whose value is the id of its associated ". In the sample HTML, there is no aria-labelledby on the tabpanel, and no id on the associated .
On the other side, there is an id on the role="tabpanel" and an aria-controls attribute on the role="tab", but they are not documented. It seems they have the same value, so I guess it's on purpose.

So:

  • Is aria-labelledby still needed ?
  • How do we know the link between the tab and the tabpanel ?
    • Through aria-labelledby and the id of its associated ?
    • Though aria-controls attribute and the id on the role="tabpanel" ?

Thanks.

@kaelig
Copy link
Contributor

kaelig commented Nov 20, 2015

Salut Fabien, merci pour le retour, tu as l'œil ! (et un grand merci pour ton récent article sur le sujet)

@stefsullrew did some great refactoring work on this component, so I'll let her take it from here.

@stefsullrew
Copy link
Contributor

Interesting. This seems to be a regression. If you look at the tabs within our own site, they're actually using the aria-labelledby attribute. But it's missing on the component. I've got to take a deeper look at this.

Meanwhile, here's a good article on a11y and tab sets @FabienTaillon - https://www.marcozehe.de/2013/02/02/advanced-aria-tip-1-tabs-in-web-apps/

@stefsullrew stefsullrew self-assigned this Nov 23, 2015
@kaelig
Copy link
Contributor

kaelig commented Nov 23, 2015

Meanwhile, here's a good article on a11y and tab sets @FabienTaillon - https://www.marcozehe.de/2013/02/02/advanced-aria-tip-1-tabs-in-web-apps/

Yep, he's the one who opened the issue :p

@stefsullrew
Copy link
Contributor

Hi @FabienTaillon, meet Fabien! Wow. I'm laughing my butt off here. At least you know you wrote a good article, right?

@FabienTaillon
Copy link
Author

Haha that's ok @stefsullrew because I did wrote a blog post but it's not this one :P
Mine is here, and talking about Lightning Design System directly:
http://www.fabientaillon.com/using-tabs-component-with-salesforce-lightning-design-system
It's while trying to update it that I found this issue.

I do have a new nice article to read though, so thanks ! :D

@stefsullrew
Copy link
Contributor

This fix will be in the next release. Thanks again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants