WyriHaximus/HtmlCompress

Not fully removing newlines with async/defer in script-tag

lifeofguenter opened this issue ยท 4 comments

    <script async defer
    src="https://maps.googleapis.com/maps/api/js?key=YOUR_API_KEY&callback=initMap">
    </script>

(via: https://google-developers.appspot.com/maps/documentation/javascript/adding-a-google-map)

Interesting, I'll have a look at it. Thanks for reporting ๐Ÿ‘

Hey sorry for the late reply, been busy with https://github.com/reactphp/http/releases/tag/v0.8.0 the past few months. I'll be adding an edge case test for this later today/tomorrow ensuring this works as intended. Thank you for reporting, I'll keep you posted ๐Ÿ‘

After digging into this, this isn't a bug because the new line takes the same amount of space as a space. As demonstrated by the test in #30 and the fix in c89bd37

Just tagged 1.3.0 with this in it ๐ŸŽ‰ .