Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mikeytown2 created an issue. See original summary.

mikeytown2’s picture

Thinking about this and this should be a sub module. This should use the `cache_advagg_info table`, hash the url and prefix it with something like `advagg:file:css-import-url:`. Use a db table in order to query info on cron to update the remote data. Updates should respect the Cache-Control headers; also allowing for a global min to be enforced.

mikeytown2’s picture

mikeytown2’s picture

User Agents

eot: Mozilla/5.0 (Windows; U; MSIE 7.0; Windows NT 6.0; en-US)
svg: Mozilla/5.0 (iPad; U; CPU OS 3_2_2 like Mac OS X; nl-nl) AppleWebKit/531.21.10 (KHTML, like Gecko) Version/4.0.4 Mobile/7B500 Safari/531.21.10
ttf: Drupal (+http://drupal.org/)
woff: Mozilla/5.0 (Windows; U; MSIE 9.0; Windows NT 9.0; en-US)
woff2: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:40.0) Gecko/20100101 Firefox/40.1

mikeytown2’s picture

mikeytown2’s picture

FileSize
4.22 KB

Starting point... not working but groundwork has be set.

mikeytown2’s picture

Technically working; still needs a lot of work but good enough to commit for now.

mikeytown2’s picture

FileSize
5.52 KB

Have the UI almost done; still need to tie it in so it actually does something

  • mikeytown2 committed e2cc1f0 on 7.x-2.x
    Issue #2827137 by mikeytown2: Add a UI for controling the import inline...
mikeytown2’s picture

FileSize
17.66 KB

This has been committed. Need to do http caching, 304 handling and some cleanup most likely.

  • mikeytown2 committed d007cae on 7.x-2.x
    Issue #2827137 by mikeytown2: Inline @import statements - better caching...
mikeytown2’s picture

mikeytown2’s picture

Status: Active » Fixed

Going to mark this as fixed.

  • mikeytown2 committed 2525caa on 7.x-2.x
    Issue #2827137 by mikeytown2: Inline external css
    
mikeytown2’s picture

Also need to do the same for external css files.

  • mikeytown2 committed be80d85 on 7.x-2.x
    Issue #2827137 by mikeytown2: Fix dns prefetch and preload for inlined...
mikeytown2’s picture

Needed to add in dns and preconnect info for the newly inlined code.

Status: Fixed » Closed (fixed)

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