search

Khan / aphrodite

Framework-agnostic CSS-in-JS with support for server-side rendering, browser prefixing, and minimum CSS generation
5.34k stars 188 forks source link

Update inject.js #357

Closed istvanmakary closed 5 years ago

istvanmakary commented 5 years ago

Based on the HTML 5 definitions, the type attribute is not required on the style tags any more.

https://www.w3schools.com/tags/att_style_type.asp

In HTML5, the type attribute is no longer required for CSS. The default value is "text/css".

Best, Istvan

kevinbarabash commented 5 years ago

I worry that this might break some older browsers that might not be fully compliant.

lencioni commented 5 years ago

Thanks for the PR! But agreed, I'm not sure this is a good trade-off to make.