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
"rowspan" attribute does not appear in DOM #264
Comments
If you do |
Oh, that's it. Thanks :) I'll await #255's resolution for long term, but adjust my code in the short term... |
Sorry @ericclemmons, it could have been :) I tried to do a pass for missing attributes pre-0.4 but if you notice anything missing from https://github.com/facebook/react/blob/master/src/dom/DefaultDOMPropertyConfig.js, feel free to open an issue or PR |
Oh, it totally looks like |
Oops! I saw colSpan and naively assumed that rowSpan would be there too… :) |
@ericclemmons Still interested in fixing this? |
Closing for #291... |
Given that this trips people up, instead of being consistent with the DOM, why not be consistent with HTML and support all-lowercase attributes? I just had to go back to the docs for this issue about the "autocomplete" attribute. I'm writing something that looks like HTML, it should work like HTML... otherwise it's an inconsistency, not consistency. |
I agree. It would be preferable (to me) if attributes were all mapped |
Fetch owners list from renderer (using suspense)
<th rowspan="2"></th>
does not setrowspan
in the DOM.Component:
Output:
Can this be my first PR? :)
The text was updated successfully, but these errors were encountered: