Add AjaxOperation framework#12
Add AjaxOperation framework#12jamsilver wants to merge 2 commits intocomputerminds:7.x-1.xfrom jamsilver:7.x-1.x
Conversation
cm_tools.patch
Outdated
There was a problem hiding this comment.
@jamsilver Why is there a patch file in the change request?
There was a problem hiding this comment.
@jamsilver Can you provide a minified version of this javascript for inclusion in production?
There was a problem hiding this comment.
I didn't make this a library / with minified options etc simply because I was following ctool's model (as the cm_tools module has been doing). Does it matter that much?
There was a problem hiding this comment.
We're better than ctools :)
Yeah, we should do this all proper like, with minified versions and hook_library implementation.
There was a problem hiding this comment.
Minified options++ Front end perf is by and large lots of small cumulative wins.
On Thursday, 8 August 2013 at 12:05, James Silver wrote:
In js/ajax-operations.js:
@@ -0,0 +1,136 @@ > +(function($) {
I didn't make this a library / with minified options etc simply because I was following ctool's model (as the cm_tools module has been doing). Does it matter that much?—
Reply to this email directly or view it on GitHub (https://github.com/computerminds/cm_tools/pull/12/files#r5654917).
So basically this adds the gubbins for making custom Ajax requests to the server from javascript without particularly being in the context of a DOM element in the way Drupal.ajax assumes.
It also adds the server-side gubbins for specifying what should happen on this ajax request and what ajax commands should be returned.
See the PHPDoc at the top of includes/ajax_operations.inc for full details of how it all works.
The use case I am in which is using this is: