Jens Oliver Meiert comments

Results 118 comments of


                                            Jens Oliver Meiert

thinking includedAutoGenerated tags should also also default to false

Hey @jayzehngebot, does this still hold? Do you have some example (before/after)? I could look at addressing this in [HTML Minifier Next](https://github.com/j9t/html-minifier-next).

collapseWhitespace remove the space around the <output> tag

(Just to make it easier for others, this was fixed in [html-minifier-terser](https://github.com/terser/html-minifier-terser) and, consequentially, [html-minifier-next](https://github.com/j9t/html-minifier-next).)

Misleading/Wrong Error Message When Detecting Invalid Custom Elements

I would agree that the message could use rephrasing (it sounds like `spoiler` could be used elsewhere), but the validator cannot know if a one-word element is supposed to be...

Add support for <script type="speculationrules">

If of interest, support for more JSON formats was just included in [HTML Minifier Next 4.6.0](https://www.npmjs.com/package/html-minifier-next) (maintained successor).

Insanely slow on large files

@Nuru, does this still persist? If yes, would pick this up for analysis in [HTML Minifier Next](https://github.com/j9t/html-minifier-next) (maintained fork).

"Collapse inline tag whitespace" doesn't work between <img> elements

Actually, the first output (for `span`) seems _incorrect_? But when testing both, e.g. ```html Test ``` the outcome seem correct for both cases even with the default settings (except with...

Any plans to minify content within script tag of type="application/ld+json"?

That should work out of the box. Can only share how it’s handled in HTML Minifier Next (enhanced successor), but [see this example in the web version](https://j9t.github.io/html-minifier-next/#N4IgliBcIDwM4GMBOYAOAXABOgnqgpgLwA6IAhqqgDZgJnpgD2AdgPRUAmA1AFZwukAfMWbARmTKQACCFunwAPdKUiSQAC3TpUcSK1Z8WAWk4A6RkgDmrWc3lK4rAkn7NTh5p1IAacWqlgHCpqmtp6rBwARgQcYGTmVqxI+PwArkgI+KwAUozqzAD6ADL4zMwCIL7MEqTMZAC2+MGkufmYJWUVVTUgkRbMzSAAjACcACwADEZDUxMjPn6kcKiMqXBNUCFaqOFRMXEJ1slpGVkAwjh26nHFpeUDICIAviIwrIgoGIKVIIxQoHR1gBlUpwMAMABu+CgADMyFR1t4QLIqFQKOsAIJaFCRVLyADq13kyzImVh8MRyMYqPR+AAQoxqfgyMwsegcXiUuSEfgkSi0Tp8ABJTxgZj4AAqZEshPBKVQpOhkDhPL51IF61lxIVZOVFN5VOY6yQEPoYChZ3VtO5lI4+FkdoAonZwWAuXrVRp0PUqABWG0GsUIKipO0YvGMADipXwSHo+A4UsscADSIA1vh8KgzlRGGDmJYgWi4OpUyB6mKwDCcNkgWWK8wqzgAKoAJSKKY9lPKADl8AB3GjiuB0-AwiyS6U5vNKlWUhVIBjwgCyZCQadSqDLqGOsahRTF9OSZDTnbnBp3+ChdjZHOJjsQFDFlm3SEYmTgcEtzFiDBY8MtepGjsM99SRZJ6kYKFbzAXF5AARVSRhiTLCCoPwQDgPQUDPTQqFHXqDAcBguD3XPcD8Eg-DCNwR0qEo0psNQyj0IAeQwJg6ioJMcMpPD8FbBNUh-Fl0BIzleINfigWQNB0AlPB8HElCuyklioSBXB6IPZg0wUghlLIsCQH4pMtXlRUy34RdDMkpFrPQHMyE-HsGlnYz2TAeozjWdBGHqAAxONLCwuyQDWfAgXUCx0AAEXfXACADJ4niAA). (I took this...

Jens Oliver Meiert