CSS aggregation fails when there's an unquoted url() with "/*" in it

Created on 29 August 2012, about 12 years ago
Updated 11 December 2023, 10 months ago

Currently the aggregator fails when it encounters a CSS like

background: url(/path/*file*)

because /* is treated as the start of CSS comment.

A proposed fix will be attached in short time. It basically keeps unquoted url() intact.

This was discovered along with a discussion about the possibility of including //-style syntax on www-style, the mailing list for CSS spec development. If //-style comment are introduced in the future, we need this patch to deal with the very general

@import url( http://dont.com/even/look.css )

too, before //-style comments are removed.

🐛 Bug report
Status

Needs work

Version

11.0 🔥

Component
CSS 

Last updated 1 day ago

Created by

🇹🇼Taiwan kennyluck

Live updates comments and jobs are added and updated live.
  • Needs tests

    The change is currently missing an automated test that fails when run with the original code, and succeeds when the bug has been fixed.

Sign in to follow issues

Merge Requests

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.71.5 2024