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

tabIndex attribute #131

Closed
fred-wang opened this issue Sep 7, 2019 · 4 comments
Closed

tabIndex attribute #131

fred-wang opened this issue Sep 7, 2019 · 4 comments
Labels
css / html5 Issues related to CSS or HTML5 interoperability MathML Core Issues affecting the MathML Core specification MathML 4 Issues affecting the MathML 4 specification need specification update Issues requiring specification changes

Comments

@fred-wang
Copy link

See #118 and w3c/mathml-core#129

We need to clearly define how the tabIndex attribute behave, giving the default value, taking into account MathML links (elements with the href attribute) etc

We also need more tests for that as we only have one basic test. More tests were added in https://bugs.webkit.org/show_bug.cgi?id=200470 but they are internal to WebKit. They also don't seem to consider interactions with MathML links.

@fred-wang fred-wang added css / html5 Issues related to CSS or HTML5 interoperability MathML 4 Issues affecting the MathML 4 specification MathML Core Issues affecting the MathML Core specification need implementation update need resolution Issues needing resolution at MathML Refresh CG meeting need tests Issues related to writing WPT tests labels Sep 7, 2019
@fred-wang
Copy link
Author

I believe https://web-platform-tests.org/writing-tests/testdriver.html should be enough to write UI tests for tabIndex similar to WebKit's internal ones.

@NSoiffer
Copy link
Contributor

NSoiffer commented Sep 9, 2019

The HTML 5.1 spec says the default value is a user agent decision when it is not specifically given

When the attribute is omitted, the user agent applies defaults

We can of course make a suggetion that all elements with href should have tabindex=0.

@fred-wang
Copy link
Author

I think this is essentially fixed by w3c/mathml-core@c2d770c where we now defer the semantic to HTML.

With this plus my previous comment, I think we can close this issue.

@fred-wang fred-wang added need specification update Issues requiring specification changes and removed need resolution Issues needing resolution at MathML Refresh CG meeting need tests Issues related to writing WPT tests labels Sep 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
css / html5 Issues related to CSS or HTML5 interoperability MathML Core Issues affecting the MathML Core specification MathML 4 Issues affecting the MathML 4 specification need specification update Issues requiring specification changes
Projects
None yet
Development

No branches or pull requests

2 participants