Problem/Motivation
The css/js query string is used when:
1. Aggregation is disabled site-wide
2. Aggregation is disabled for an individual CSS file
3. Aggregation is disabled for an individual JavaScript file and the library for that file doesn't have a version.
One most sites, none of these will be very common in production, so we can slightly refactor some logic to ensure it's only requested at the point it's actually needed.
Steps to reproduce
Proposed resolution
Remaining tasks
User interface changes
Introduced terminology
API changes
Data model changes
Release notes snippet
Issue fork drupal-3571351
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
- 3571351-only-get-the
changes, plain diff MR !14625
Comments
Comment #2
catchComment #4
smustgrave commentedEye test LGTM. Guess proof is in the pudding with the performance improvements
Comment #5
longwaveIf it wasn't for the performance tests we could backport this down to 11.3.x and 10.6.x. Open to doing that if anyone wants to open a fresh MR? Otherwise, let's just leave this until 11.4.0.
Committed and pushed 3c07b176ed1 to main and 7f0aa96ad4c to 11.x. Thanks!
Comment #10
catchIt's a very small improvement so I think it's fine for it to wait until 11.4.