Right. Sorry, I figured that out a bit after I hit send. I figured I'd add
that to lighthouse after it got created...

On Wed, Apr 6, 2011 at 9:05 AM, Arian Stolwijk <[email protected]> wrote:

> `this.get('formRequest')` should get the Form.Request instance, which is
> correct, however `Element.Properties.formRequest` is broken, and also you
> can only do `this.get('formRequest')` because it doesn't accept multiple
> arguments, like is used in formUpdate.
>
>
>
>
> On Wed, Apr 6, 2011 at 5:53 PM, Aaron Newton <[email protected]> wrote:
>
>> A couple of things.
>>
>>    - you are setting options on the request, so you need to indent your
>>    options object for that per the docs
>>    eg.
>>    form.set('send',{
>>        requestOptions: {
>>            useSpinner: true,
>>            spinnerTarget: form
>>        }
>>    });
>>    - You also need to tell it what to update (form.formUpdate(target));
>>    You can also do form.formUpdate(target, options);
>>    - jsfiddle's echo/json needs you to give it some data; not really the
>>    issue here but may prevent you from seeing a result
>>    - finally, you've found a bug!
>>
>> Check it out:
>>
>>
>> https://github.com/mootools/mootools-more/blob/master/Source/Forms/Form.Request.js#L214
>>
>> You'll note that on line 188 and 204 (and elsewhere) we store the instance
>> on the element as "form.request" but on line 214 it's "formRequest" so it
>> doesn't retrieve it. So element.formUpdate is broken.
>>
>> For the time being, you can just create the instance the regular way (var
>> fq = new Form.Request(form, updateElement, options)).
>>
>> Would you like to open a lighthouse ticket for this bug? (just paste in
>> this last part). I can do it, but if you do it you'll get emails as we work
>> on it and resolve it.
>>
>> https://mootools.lighthouseapp.com/projects/24057-mootoolsmore
>>
>> Aaron
>>
>> On Tue, Apr 5, 2011 at 8:41 PM, dukeofgaming <[email protected]>wrote:
>>
>>> Hi,
>>>
>>> Quick and perhaps dumb question. Why wont the Request spinner
>>> parameters work when set from the send property?.
>>>
>>> http://jsfiddle.net/dukeofgaming/neQ3X/5/
>>>
>>> I'd swear I've made it work like this, am I missing something?
>>>
>>> Thanks in advance
>>>
>>
>>
>

Reply via email to