In aggregated CSS files paths in url() statements are not surrounded by quotes -> url(""). This way, if a path contains spaces, the file (usually a background image) can't be loaded by the browser.

CommentFileSizeAuthor
#1 798826.patch391 bytesStandart

Comments

Standart’s picture

Standart commented
Status: Active » Needs review
StatusFileSize
new 798826.patch391 bytes

Patch attached.

Status: Needs review » Needs work

The last submitted patch, 798826.patch, failed testing.

damien tournoud’s picture

damien tournoud commented

Why not, but spaces in URLs are *not* valid. You should use '+' or '%20' (depending on the part of the URL the space is in) for this.

Standart’s picture

Standart commented

The problem appeared on my dev machine, where I don't care too much how I name my folders. So I thought it would help in such scenarios without hurting anyone.

Standart’s picture

Standart commented
Status: Needs work » Needs review

#1: 798826.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, 798826.patch, failed testing.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Component: browser system » base system

When actually using + or %20 instead of spaces, you don't need quotes.

Hence, I suspect we're missing a urlencode() somewhere instead.

PS: Who's able to remove that "browser system" component? :)

Status: Needs work » Closed (outdated)

Automatically closed because Drupal 7 security and bugfix support has ended as of 5 January 2025. If the issue verifiably applies to later versions, please reopen with details and update the version.