Opened 2 years ago

Closed 2 years ago

Last modified 2 years ago

#33446 closed New feature (fixed)

ManifestStaticFilesStorage doesn't update CSS source map references

Reported by: Adam Johnson Owned by: Adam Johnson
Component: contrib.staticfiles Version: dev
Severity: Normal Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

A follow-up to #32383, which added JS source map support.

The source map spec also covers CSS source maps. I think Django should update these references too for the same reasons as JS source maps.

Change History (4)

comment:1 by Mariusz Felisiak, 2 years ago

Owner: changed from nobody to Adam Johnson
Status: newassigned
Triage Stage: UnreviewedAccepted

comment:2 by Mariusz Felisiak, 2 years ago

Has patch: set
Triage Stage: AcceptedReady for checkin

comment:3 by Mariusz Felisiak <felisiak.mariusz@…>, 2 years ago

Resolution: fixed
Status: assignedclosed

In dc8bb35:

Fixed #33446 -- Added CSS source map support to ManifestStaticFilesStorage.

comment:4 by Mariusz Felisiak <felisiak.mariusz@…>, 2 years ago

In b8113644:

Refs #33446 -- Allowed variable whitespace in CSS source map references.

Follow up to dc8bb35e39388d41b1f38b6c5d0181224e075f16.

The Webpack default is to output CSS source map comments like
/*# sourceMappingURL=main.css.map*/. Also, Chromium allows tabs.

Note: See TracTickets for help on using tickets.
Back to Top