-
-
Notifications
You must be signed in to change notification settings - Fork 401
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
Validation failure: The “type” attribute for the “style” element is not needed and should be omitted #724
Comments
didn't know adding optional type attribtute is an antipattern now |
You can reproduce the warning on your documentation website with https://validator.w3.org/: |
It should be as simple as removing this line: jss/src/renderers/DomRenderer.js Line 309 in 602e325
Then, maybe updating the tests and updating the docs. @mikkopori Do you want to work on it? |
It would be fair since I reported this :) Never worked on this before, but I guess I'll learn quick. It will take a couple of days before I have time to look into this, so if anyone is faster feel free. |
released 9.8.6 |
Hello,
testing my application with a validator I ran into these warnings:
The “type” attribute for the “style” element is not needed and should be omitted
for example in
<style type="text/css" data-jss="" data-meta="MuiToolbar"> .. </style>Since I noticed that validation errors were reported here earlier I thought I should report this as well. Did not find any mention of this particular warning.
:)
The text was updated successfully, but these errors were encountered: