Add support for rel=preload - #951

Updated the test suite for #931 and #926.

Hard-coded hashes updated by working through failing unit tests one by one... maybe there was an easier way?

Cheers!

I cherry-picked the second commit into #952 and merged that. thanks!

regarding the first commit, i don't fully understand the matter, but it seems there are other attributes besides preload that could be supported, see #690. if that's the case, it seems like your solution does not scale to multiple attributes because you'd need one OUTPUT_MODE per attribute combination, is that correct?

Hi karyon,

rel=preload is more like a directive than an attribute: it is intended to give the browser a heads up that a resource will be needed, so start loading it now. The purpose of adding a preload output mode to django-compressor is to be able to take advantage of this directive for our own cached resources.

With this additional output mode, django-compressor can now generate an inline compressed script/stylesheet, an external compressed file, or simply the preload tag (which does the same thing as file but returns the preload tag instead of the script/link tag). It really is unrelated to the problem of extra attributes.

My workflow with this version of django-compressor uses preload mode in the head and then calls the actual scripts as needed. preload really just takes care of generating the correct filename in the tag: instead of <script src="..."> in the head, it will output <link rel="preload" as="script" href="...">.

<head>
...
{% compress js preload %}
<script src="{% static 'website/js/index.min.js' %}"></script>
{% endcompress %}
...
</head>

<body>
...
{% compress js %}
<script src="{% static 'website/js/index.min.js' %}"></script>
{% endcompress %}
...
</body>

Cheers!

thanks!

Any idea when this is going to be released?

no, sorry.

(fwiw, it's really really easy to put a specific commit of a git repository into your requirements.txt, try it! :D there's no additional testing or the like done on releases, so there's little downside for you.)

I'm planning to do a release later this month. Please use the develop branch in the meantime.

Nice, but it seems that it's not working when COMPRESS_ENABLED is False, then it juste leaves the js files ready to be executed :/

@VivienGiraud can you please open a new issue with the actual and expected behavior?

@karyon Thanks, here is the link : #987