Issue

While investigating #2915654: Deferred CSS feature does not take media attribute into consideration, I discovered (remember/read again) that the regex for CSS Defer excludes external sites.
Ideally external sites should be included as well.

Solution

- Add option to enable deferring external CSS as well.
- Set config to be on by default but leave disabled for existing sites when upgrading so as to avoid causing any regressions on existing sites.
- update documentation

Comments

NickWilde created an issue. See original summary.

nickdickinsonwilde’s picture

nickdickinsonwilde
He/Him/His
Primary language English
Location Victoria, BC (T'So-uke lands)
commented
Assigned: nickdickinsonwilde » Unassigned
Status: Active » Fixed

  • NickWilde committed 31b8d70 on 8.x-3.x 
    Issue #2916944 by NickWilde: CSS Defer functionality doesn't affect...
yohan31’s picture

yohan31 commented

Nick,

I believe the same is true for 7.x-2.28. I have one JS file on my site, and I cannot get it to set that script to defer. Could you look at the 7.x branch and see if you can apply this change there as well?

Status: Fixed » Closed (fixed)

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