Make compatibility configurable about autoptimize HOT 8 CLOSED

Additionally two things :)

1. I think autoptimizeCompatibility class should add exclusion to both:
   jquery.min.js & jquery.js

2. Also it would be best if this could recognize DOMContentLoaded jquery scripts (probably also window load event?):

<script type="text/javascript">document.addEventListener("DOMContentLoaded", function () {
jQuery(function($) { 
// something
});
});
</script>

Those are executions that will happen after jQuery is loaded - so it could be minifed.

from autoptimize.

from autoptimize.

Yes, I think that makes sense.

from autoptimize.

OK, took me some time (I'm slow on Mondays) but I think I implemented this with 1a8ec3d and e538520. The domcontentloaded one is a bit more tricky to get right though, so I'll probably "forget" about that (unless you would have a great PR for that ;-) ). Would be great if you could test these changes?

from autoptimize.

have you been able to have a look at those changes/ test them by any chance @fliespl

from autoptimize.

Yeah, I have tried that, but it really is more complicated :) Especially since items can be mixed (domcontentloaded function closed, while other jquery is appended after), I think it might end up with plenty of issues so probably worth dropping this idea.

from autoptimize.

It indeed is impossible to take all variations into account, but also auto-excluding jquery.js (which is pretty harmless) and allowing pre-3.0-installed instances to enable the compatibility logic does have some merit so I'll probably keep those in :-)

thanks!

from autoptimize.

Oh yes - those changes are cool and work :) Was simply talking about domcontentloaded.

Thanks!

from autoptimize.