Feature Request: Disable minify for admins or logged-in users!

An admin doesn’t “need” minified HTML, but what’s the harm? Also, by having HTML minified for admins it allows you to easily test that it’s working correctly.

I’m just at a loss as to what the problem is with the admin HTML being minified. If it’s readability, simply turn it off for development or use browser development tools to review the source.

Admins can test it easily in incognito mode for a real non-logged-in user. Please provide this functionality because we view-source often to check that our themes are working. We shouldn’t need to disable minify during testing and then re-enable it. It’s cumbersome for people who’re in active development mode while the site is still live. Listen to your customers.

It’s not that I’m not listening, it’s that not minifying for logged in users would make all content non-minified for sites where most users are logged in. Also, I thought it wasn’t minifying at all for you? But now it’s minifying too much? Which is it? Finally, this is a free plugin, with no money compensation. I programmed it and released it for free. There are no “customers” to listen to. If this was a paid plugin, and you paid, then you would be a customer. As it is now, you should be thankful for getting something for free. But, if you’d like to go to a paid model, I’m willing to add this feature for paying customers. But seriously, you’re being unreasonable and offensive to my hard work I’ve given you for free.

• This reply was modified 7 years, 9 months ago by Tim Eckel.
• This reply was modified 7 years, 9 months ago by Tim Eckel.
• This reply was modified 7 years, 9 months ago by Tim Eckel.
• This reply was modified 7 years, 9 months ago by Tim Eckel.

Yes it’s not minifying for me. The smart way to implement this, if you’re listening, is to allow disabling minifying as an option. Then a site admin can choose what to do. To accommodate sites that have many logged in users it could be allowed for role types, such as admin.

Yup, that would be a way of doing it, I just don’t see the need. When using developer tools, minification doesn’t matter anyway, so I see no point in making this an option when it doesn’t make a bit of difference when looking at your site markup in the developer tools console.

I would suggest you figure out why it’s NOT working for you before you ask for a new feature that would make zero difference at this point to you.

And by the way, Minify HTML already doesn’t minify if admin, see one of the first lines of the source:

if ( !is_admin() ) add_action( ‘init’, ‘teckel_init_minify_html’, 1 );

Basically, you can’t even build a theme that works and you’re asking for features that already exist.

I was suggesting a good option. The matter is unrelated to why my theme is not monifying. Tell you what. Keep the plugin as is. Clearly I’m not the only one wanting a feature. There are easily available compression functions which I shall use with an if condition to check for user. Thank you.

Why are you asking for a feature when you can’t get the plugin working at all? Very confused. Basically, if you got the plugin working, you would realize that you don’t need this “feature” as there’s no need for admins to see non-minified markup as developers use browser developer tools, not view source. Also, the plugin already doesn’t minify for admin view. You’re putting your cart before the horse.

• This reply was modified 7 years, 9 months ago by Tim Eckel.

Hi all,

I am marking it as resolved (initially, I kept it as “not a support question”). Please seem confused on what’s going on.

Thanks all.

Pothi