-
Notifications
You must be signed in to change notification settings - Fork 102
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
Converts attributes to lower case #45
Comments
I have a same issue with inline styles |
This updates the list of react attributes, including things like SVG, and also removes the down-conversion to lower-case that previously used to be done.
Update list of React attributes. Fixes peternewnham#45.
Lowercases the tags name as well??
|
and my |
I threw a workaround over in #56 |
Same here, what I did is adding an attribute viewBox with viewbox's value. If someone has a better a solution... |
I ended up skipping this library altogether, there were too many issues with the SVG, and just setting the content like: |
I've used svg-to-jsx library to convert first to jsx and then run it by this one. All of the properties are read correctly except "preserveaspectratio" which is reverted back to lowercase. |
Same for "textarea" tag name, it does hit up a warning to use "defaultValue" attribute, but impossible to get "defaultValue", instead it is "defaultvalue" after parsed. Do someone know an other html parser library still maintained and not to much bugged ? |
Did you issue get fixed? I am facing the same issue now. |
is that issue fixed? still i got the lowercase issue |
If I bind onChange on input tag, it converts to lowercase and ReactJS gives a warning.
Can you fix this?
The text was updated successfully, but these errors were encountered: