Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
kurapovCreditAttribution: kurapov as a volunteer commented
Status:
Closed (fixed)
» Needs work
I find the option's title 'Use link rel="preload"' highly misleading - it doesn't add the attribute to CSS resources nor in any way influences the support for preload functionality built into most modern user agents, merely loads a polyfill for legacy/IE browsers so may I propose a less hair-pulling title like 'Load polyfill for legacy browser support of rel="preload" attribute'?
Agreed. I was expecting the rel="preload" to be added to the main css files.
Is that something that will be fixed, or is the feature expected to work this way?
Thanks for the heads up on this. This patch has been committed. Leaving it as needs work as I need to translate the checkbox setting over the the radio options in an update.
Comments
Comment #3
mikeytown2 CreditAttribution: mikeytown2 commentedFollowing patch has been committed
Comment #5
kurapov CreditAttribution: kurapov as a volunteer commentedI find the option's title 'Use link rel="preload"' highly misleading - it doesn't add the attribute to CSS resources nor in any way influences the support for preload functionality built into most modern user agents, merely loads a polyfill for legacy/IE browsers so may I propose a less hair-pulling title like 'Load polyfill for legacy browser support of rel="preload" attribute'?
Comment #6
baptisten CreditAttribution: baptisten commentedAgreed. I was expecting the rel="preload" to be added to the main css files.
Is that something that will be fixed, or is the feature expected to work this way?
Comment #8
mikeytown2 CreditAttribution: mikeytown2 commentedThanks for the heads up on this. This patch has been committed. Leaving it as needs work as I need to translate the checkbox setting over the the radio options in an update.
Comment #10
mikeytown2 CreditAttribution: mikeytown2 commentedCommitted this update. Marking as fixed.