Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mikeytown2 created an issue. See original summary.

  • mikeytown2 committed 0048a15 on 7.x-2.x
    Issue #2660710 by mikeytown2: Add in css preload attribute support.
    
mikeytown2’s picture

Status: Active » Fixed
FileSize
12.69 KB

Following patch has been committed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

kurapov’s picture

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'?

baptisten’s picture

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?

  • mikeytown2 committed b68faca on 7.x-2.x
    Issue #2660710 by mikeytown2: Add in support for the preload attribute
    
mikeytown2’s picture

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.

  • mikeytown2 committed c6bcd0d on 7.x-2.x
    Issue #2660710 by mikeytown2: Migrate the preload checkbox into the...
mikeytown2’s picture

Status: Needs work » Fixed
FileSize
1.36 KB

Committed this update. Marking as fixed.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.