Code in seperate css file :

.circle-background{background-image:url("../../assets/images/PC_slices/形状结合@2x(1).png"),url("../../assets/images/PC_slices/形状结合备份@2x.png");

After merge:

.circle-background{background-image:url(/profiles/custom/laulawyer/themes/custom/laulawyer_tip_off/assets/images/PC_slices/%E5%BD%A2%E7%8A%B6%E7%BB%93%E5%90%88%402x%281).png"),url(/profiles/custom/laulawyer/themes/custom/laulawyer_tip_off/assets/images/PC_slices/%E5%BD%A2%E7%8A%B6%E7%BB%93%E5%90%88%E5%A4%87%E4%BB%BD%402x.png);

We notice that the aggregation process treat multi urls as a single url value, there is a redundant quotation mark in the end of the first url value.

This cause the style code after that quotation mark lose efficacy.

Comments

司南 created an issue. See original summary.

司南’s picture

司南 commented
Issue summary: View changes
司南’s picture

司南 commented
Issue tags: +CSS, +aggregation, +merge, +urls, +background-image, +url, +quotation mark

Version: 9.2.x-dev » 9.3.x-dev

Drupal 9.1.10 (June 4, 2021) and Drupal 9.2.10 (November 24, 2021) were the last bugfix releases of those minor version series. Drupal 9 bug reports should be targeted for the 9.3.x-dev branch from now on, and new development or disruptive changes should be targeted for the 9.4.x-dev branch. For more information see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

Version: 9.3.x-dev » 9.4.x-dev

Drupal 9.3.15 was released on June 1st, 2022 and is the final full bugfix release for the Drupal 9.3.x series. Drupal 9.3.x will not receive any further development aside from security fixes. Drupal 9 bug reports should be targeted for the 9.4.x-dev branch from now on, and new development or disruptive changes should be targeted for the 9.5.x-dev branch. For more information see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Component: cache system » asset library system
Issue tags: -CSS, -aggregation, -merge, -urls, -background-image, -url, -quotation mark +Bug Smash Initiative

Reassigning to asset library component, as that is where CSS aggregation issues belong. I looked for duplicates but couldn't find an exact match. This probably needs a test case writing first and then a fix can be found from there.

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Issue tags: +CSS aggregation
nod_’s picture

nod_
Primary language French
Location Lille
commented

giving strong vibes of related #2936067: CSS aggregation fails on many variations of @import

I think the regex to fix is here: https://git.drupalcode.org/project/drupal/-/commit/7374845dcbbe31e24244b...

Or we could also look at having a proper css parser in a followup to avoid having to mess with regexes :p

Version: 9.4.x-dev » 9.5.x-dev

Drupal 9.4.9 was released on December 7, 2022 and is the final full bugfix release for the Drupal 9.4.x series. Drupal 9.4.x will not receive any further development aside from security fixes. Drupal 9 bug reports should be targeted for the 9.5.x-dev branch from now on, and new development or disruptive changes should be targeted for the 10.1.x-dev branch. For more information see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

Version: 9.5.x-dev » 11.x-dev

Drupal core is moving towards using a “main” branch. As an interim step, a new 11.x branch has been opened, as Drupal.org infrastructure cannot currently fully support a branch named main. New developments and disruptive changes should now be targeted for the 11.x branch. For more information, see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

Version: 11.x-dev » main

Drupal core is now using the main branch as the primary development branch. New developments and disruptive changes should now be targeted to the main branch.

Read more in the announcement.