Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

collectstatic doesn't work properly with new version of jquery-ui #311

Closed
pecheneff opened this issue Dec 21, 2023 · 0 comments
Closed

collectstatic doesn't work properly with new version of jquery-ui #311

pecheneff opened this issue Dec 21, 2023 · 0 comments
Assignees

Comments

@pecheneff
Copy link

pecheneff commented Dec 21, 2023

Hello.

There is an issue with the new version of jquery-ui. When running collectstatic management command and using ManifestStaticFilesStorage as static files storage the command throws an error.

Screenshot 2023-12-21 at 02 30 54

The problem is jQuery authors put a comment to jquery-ui-min.css file which contains strings like url(%22images%2Fui-icons_555555_256x240.png%22). So when Django parses this file it tries to use these CSS url() statements to build static paths.

Removing of this comment from the file fixes the issue.

Please remove the comment which contains CSS url() statements since this comment is not compatible with Django's collectstatic. Thanks in advance.

There is a ticket in Django related to this issue.

@crucialfelix crucialfelix self-assigned this Dec 21, 2023
crucialfelix added a commit that referenced this issue Jan 10, 2024
fix #311 - Django's ManifestStaticFilesStorage fails if there are css urls in the comments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants