JS
http://drupal.org/project/jquery_update
http://drupal.org/project/labjs
http://drupal.org/project/headjs
http://drupal.org/project/librejs
http://drupal.org/project/scriptjs
http://drupal.org/project/omega - omega_get_js
http://drupal.org/project/adaptivetheme - adaptivetheme_js_alter()
http://drupal.org/project/arctica - arctica_js_alter()
http://drupal.org/project/aurora - aurora_template_process_html_override()
http://drupal.org/project/mojo - mojo_js_alter()
http://drupal.org/project/mothership - mothership_js_alter()
http://drupal.org/project/uglifyjs
http://drupal.org/project/async_script_shim
CSS
http://drupal.org/project/css_emimage
http://drupal.org/project/cdn
color module
http://drupal.org/project/precision - precision_process_html()
http://drupal.org/project/zentropy - zentropy_process_html()
http://drupal.org/project/adaptivetheme - adaptivetheme_css_alter()
http://drupal.org/project/arctica - arctica_js_alter()
http://drupal.org/project/mothership - mothership_css_alter()
Other
http://drupal.org/project/stage_file_proxy
http://drupal.org/project/magic
Comment | File | Size | Author |
---|---|---|---|
#15 | advagg-1946218-15-check-stage-file-proxy.patch | 1.48 KB | mikeytown2 |
#14 | advagg-1946218-14-fix-js-aggregation.patch | 1.14 KB | mikeytown2 |
#12 | advagg-1946218-12-labjs-patch-check.patch | 863 bytes | mikeytown2 |
#7 | advagg-1946218-7-css_emimage-fixes.patch | 6.76 KB | mikeytown2 |
#6 | advagg-1946218-6-css_emimage-fixes.patch | 1.34 KB | mikeytown2 |
Comments
Comment #0.0
mikeytown2 CreditAttribution: mikeytown2 commentedUpdated issue summary.
Comment #0.1
mikeytown2 CreditAttribution: mikeytown2 commentedUpdated issue summary.
Comment #0.2
mikeytown2 CreditAttribution: mikeytown2 commentedUpdated issue summary.
Comment #0.3
mikeytown2 CreditAttribution: mikeytown2 commentedupdate
Comment #1
mikeytown2 CreditAttribution: mikeytown2 commentedComment #1.0
mikeytown2 CreditAttribution: mikeytown2 commentedUpdated issue summary.
Comment #1.1
mikeytown2 CreditAttribution: mikeytown2 commentedUpdated issue summary.
Comment #1.2
mikeytown2 CreditAttribution: mikeytown2 commentedUpdated issue summary.
Comment #1.3
mikeytown2 CreditAttribution: mikeytown2 commentedUpdated issue summary.
Comment #1.4
mikeytown2 CreditAttribution: mikeytown2 commentedUpdated issue summary.
Comment #1.5
mikeytown2 CreditAttribution: mikeytown2 commentedUpdated issue summary.
Comment #1.6
mikeytown2 CreditAttribution: mikeytown2 commentedUpdated issue summary.
Comment #2
mikeytown2 CreditAttribution: mikeytown2 commentedCreated this issue #1968998: Get a 7.x version out that works with Advagg 7.x-2x
Comment #3
infines CreditAttribution: infines commentedI'm not seeing any issues with Labjs, Headjs, and Librejs.
Comment #4
mikeytown2 CreditAttribution: mikeytown2 commentedThanks for the report :)
Following patch has been committed.
Comment #5
mikeytown2 CreditAttribution: mikeytown2 commentedFollowing patch has been committed.
Comment #6
mikeytown2 CreditAttribution: mikeytown2 commentedFollowing patch has been committed.
Comment #7
mikeytown2 CreditAttribution: mikeytown2 commentedFollowing patch has been committed.
Comment #8
mikeytown2 CreditAttribution: mikeytown2 commentedJust posted a patch #1942230-6: CDN module AdvAgg 7.x-2.x Integration
Comment #9
infines CreditAttribution: infines commentedNothing to report with scriptjs either.
Just a note: I'm not going to test with jquery update as I've personally never had any success with jquery update, so if someone could tackle that module it would be best.
Comment #9.0
infines CreditAttribution: infines commentedUpdated issue summary.
Comment #10
mikeytown2 CreditAttribution: mikeytown2 commentedI have advagg working with jquery update. It was one of the first ones I tested.
Comment #10.0
mikeytown2 CreditAttribution: mikeytown2 commentedstriking tested modules
Comment #10.1
infines CreditAttribution: infines commentedjquery cross off
Comment #11
mikeytown2 CreditAttribution: mikeytown2 commentedI just tested LABjs and it's not compatible. The JS that gets loaded is not from AdvAgg.
Comment #12
mikeytown2 CreditAttribution: mikeytown2 commentedCreated this issue for LAB.js and I've committed this patch which will check to make sure the patch has been applied to labjs on the status report page.
#1977122: AdvAgg 7.x-2.x Integration
Comment #12.0
mikeytown2 CreditAttribution: mikeytown2 commentedUpdated issue summary.
Comment #12.1
mikeytown2 CreditAttribution: mikeytown2 commentedUpdated issue summary.
Comment #13
mikeytown2 CreditAttribution: mikeytown2 commentedCreated a Stage File Proxy issue: #1977170: AdvAgg 7.x-2.x Compatibility
Comment #13.0
mikeytown2 CreditAttribution: mikeytown2 commentedUpdated issue summary.
Comment #13.1
mikeytown2 CreditAttribution: mikeytown2 commentedUnsure about the other modules as labjs was tested as good but I found that it didn't work without patches.
Comment #13.2
mikeytown2 CreditAttribution: mikeytown2 commentedUpdated issue summary.
Comment #13.3
mikeytown2 CreditAttribution: mikeytown2 commentedUpdated issue summary.
Comment #13.4
mikeytown2 CreditAttribution: mikeytown2 commentedUpdated issue summary.
Comment #14
mikeytown2 CreditAttribution: mikeytown2 commentedFollowing patch has been committed
Comment #14.0
mikeytown2 CreditAttribution: mikeytown2 commentedUpdated issue summary.
Comment #15
mikeytown2 CreditAttribution: mikeytown2 commentedFollowing patch has been committed
Comment #15.0
mikeytown2 CreditAttribution: mikeytown2 commentedUpdated issue summary.
Comment #16
mikeytown2 CreditAttribution: mikeytown2 commentedMarking this issue as fixed. Will field one offs as they come in.
Comment #17.0
(not verified) CreditAttribution: commentedUpdated issue summary.
Comment #18
shawnrosspeters CreditAttribution: shawnrosspeters commentedAny chance of figuring out why Mothership themes do not place nice with AdvAgg?
http://drupal.org/project/mothership - mothership_js_alter()
http://drupal.org/project/mothership - mothership_css_alter()
I've successfully gotten AdvAgg and a Mothership subtheme to play nice, EXCEPT, contextual links are NOT output. The weird part is that the ul for the contextual links is output, but the
<a class="contextual-links-trigger" href="#">Configure</a>
is not output rendering contextual links virtually useless. I've also checked Chrome inspector and there's no errors of any kind output so I'm at a loss...The craziest part, and why I want to figure out a way to get this to work, is that with Mothership's options for "Compress all javascripts into one file" and "One css file!" enabled, and AdvAgg enabled the website receives between 97 and 99 score on Google PageSpeed Insights, and the load time on Pingdom is between 800ms and 900ms with less than 20 total http requests.
If the Mothership theme, and AdvAgg could play nice together then we could have some INSANE performance for Drupal all without Boost, Varnish or any other performance boosters. I'd "hate" to see what adding something like Varnish to the mix would do. Maybe a <=500ms load time, and way more potential traffic to be handled? Oh be still my heart.... :-)
Comment #19
shawnrosspeters CreditAttribution: shawnrosspeters commentedMarking this as Active.
Comment #20
mikeytown2 CreditAttribution: mikeytown2 commentedI'm pretty sure Mothership's "Compress all javascripts into one file" (mothership_js_onefile) and "One css file" (mothership_css_onefile) are redundant if using AdvAgg's "Combine CSS files by using media queries checkbox". I would also bet that mothership_js_jquery_latest & mothership_js_jquerycdn can be replaced by https://drupal.org/project/jquery_update thus bringing jquery UI up to date as well.
I'll test out the mothership theme some time in the near future.
Depending on how your CSS/JS is added you might see some benefits with advagg_mod's "Optimize CSS/JavaScript Ordering" options. If you have "aggregate" "inline" "aggregate" "inline" these checkboxes should help to group the aggregates together more (to see an example of this, view the source of drupal.org, you will see the inline gaq code in-between js aggregates followed by inline js for drupal.settings).
Comment #21
mikeytown2 CreditAttribution: mikeytown2 commented@shawnrosspeters
After doing some testing, that is an issue with mothership not advagg #2061657: Contextual Links not working after upgrade from 2.8 to 2.10
Marking this issue back to closed (fixed)
Comment #22
mikeytown2 CreditAttribution: mikeytown2 commentedKeeping this here so I can track it.
#2269885: Incompatibility with advagg
Comment #23
shawnrosspeters CreditAttribution: shawnrosspeters commented@mikeytown2
It looks like Mothership 7.x-3.x makes overriding Contextual Links CSS & JS optional. There's a new area labeled "Override Contextual Links" with the description:
"Override default contextual links CSS and JS".
I updated the based Mothership theme to version 3 and voila, contextual links return and I can use mothership and AdvAgg together. All is right in the world.
:::UPDATE:::
For those having issues upgrading to 7.x-3.x, you can modify 7.x-2.x to get the contextual links back. What I did was comment out lines 7-14 from "mothership/functions/js.php". Probably not the BEST solution but for a mature project, it works for now. I'll be upgrading my start sub-theme to use Mothership 7.x-3.x but at least I can get Mothership and AdvAgg playing nice.
Comment #24
mikeytown2 CreditAttribution: mikeytown2 commented