[Django] #34563: ManifestStaticFilesStorage support for CSS module scripts in Javascript files.
Django
files.
-------------------------------------+-------------------------------------
Reporter: Michael | Owner: nobody
Type: New | Status: new
feature |
Component: | Version: 4.2
contrib.staticfiles | Keywords: Manifest Static
Severity: Normal | Files Storage, javascript css html
Triage Stage: | module scripts
Unreviewed | Has patch: 0
Needs documentation: 0 | Needs tests: 0
Patch needs improvement: 0 | Easy pickings: 0
UI/UX: 0 |
-------------------------------------+-------------------------------------
CSS modules scripts are becomming a thing, see https://web.dev/css-module-
scripts/#:~:text=CSS%20module%20scripts%20are%20available,Safari%20is%20not%20yet%20available.
Current we only support JavaScript modules and not CSS modules with the
ManifestStaticFilesStorage.
The following import paths in JS files files will need to be converted
when collecting static with ManifestStaticFilesStorage:
{{{
import sheet from './styles.css' assert { type: 'css' };
}}}
Additionally WHATWG is firming up HTML modules, so we could proactively
handle this too while we are at it:
{{{
import content from './template.html' assert { type: 'html };
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/34563>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
Django
files.
-------------------------------------+-------------------------------------
Reporter: Michael | Owner: nobody
Component: contrib.staticfiles | Version: 4.2
Severity: Normal | Resolution:
Keywords: Manifest Static | Triage Stage:
Files Storage, javascript css | Someday/Maybe
html module scripts |
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
* cc: Adam Johnson (added)
* stage: Unreviewed => Someday/Maybe
Comment:
I think it's too early for this as Firefox and Safari still don't support
it, and there doesn't seem to be any ongoing work on supporting it in the
last 2 years.
--
Ticket URL: <https://code.djangoproject.com/ticket/34563#comment:1>
Django
files.
-------------------------------------+-------------------------------------
Reporter: Michael | Owner: nobody
Type: New feature | Status: new
Component: contrib.staticfiles | Version: 4.2
Severity: Normal | Resolution:
Keywords: Manifest Static | Triage Stage:
Files Storage, javascript css | Someday/Maybe
html module scripts |
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Comment (by Adam Johnson):
I agree, let’s get multiple browser support before modifying Django. You
can implement support in your project or a third party package beforehand.
--
Ticket URL: <https://code.djangoproject.com/ticket/34563#comment:2>
Django
files.
-------------------------------------+-------------------------------------
Reporter: Michael | Owner: nobody
Type: New feature | Status: new
Component: contrib.staticfiles | Version: 4.2
Severity: Normal | Resolution:
Keywords: Manifest Static | Triage Stage:
Files Storage, javascript css | Someday/Maybe
html module scripts |
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
This package I've been working on, and hoping to get feedback on, handles
the assert syntax and will change the path of css/html files in an import
statement, https://github.com/blighj/django-manifeststaticfiles-enhanced
--
Ticket URL: <https://code.djangoproject.com/ticket/34563#comment:3>
Django
files.
-------------------------------------+-------------------------------------
Type: New feature | Status: assigned
Severity: Normal | Resolution:
Keywords: Manifest Static | Triage Stage:
Files Storage, javascript css | Someday/Maybe
html module scripts |
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
* owner: nobody => blighj
* status: new => assigned
Comment:
https://github.com/django/django/pull/19561 Will solve this ticket too.
This ticket can be closed if that is approved
--
Ticket URL: <https://code.djangoproject.com/ticket/34563#comment:4>