Tim Eckel

If you don’t want to remove code comments, simply turn off that option. It doesn’t seem required to make it 3 options for your one use case when no one else has ever had an issue.

Remove relative domain from internal URLs is set to OFF by default, so your fears are unfounded.

I’ve never used OGP tags and probably others as well. There’s other situations where a setting causes a problem. The solution, instead of trying to make unique solutions for every possible 3rd party tool or plugin, it so simply create a switch to turn a certain feature off, which is currently implimented. The complexity is the need for every other unique case to also be addressed, not just OGP tags (that’s just one of many).

The solution is to simply turn off remove relative domain from internal URLs.

Good catch, I’m not using 8.1 and didn’t know this was deprecated. I’ll look into this and resolve the issue. It is just a warning, so it continues to be operational.

It’s not a bug and it doesn’t “break” CSS style. What it does is uncover assumptions made in style that rely on white space, which could be considered template debugging. From the plugin documentation FAQs:

Normally, you won’t notice any changes to the way the site looks in the browser. However, if you rely on white space between HTML tags (which you shouldn’t be doing) you may notice very isolated elements with reduced horizontal spacing. This is because inline objects will add a few pixels of spacing if there’s whitespace between the objects. I consider Minify HTML a good way of finding these style ‘errors’ and correcting them with proper padding or margin style.

So what you consider a bug, I consider a feature to expose style bugs.

@villekautto @jirsbek @byothe @gringopost @sujaljain007 @hgaugenot

New version released which should resolve this issue.

@jirsbek Thanks.

@gringopost Are you also removing this part of the code in the plugin?

if ( !is_admin() ) {
	if ( !( defined( 'WP_CLI' ) && WP_CLI ) ) {
		add_action( 'init', 'teckel_init_minify_html', 1 );
	}
}

As if you’re not, it’s calling teckel_init_minify_html() twice.