Hi Hippyjim Starbrook,
this is not the way you should do. It's better to use a wrapper to t
(he native function. First because you did not modify the standard
behaviour of prototype, and also because you still have the capability
to call the original function.
Have a look to the wrap() function at
http://api.prototypejs.org/language/function.html#wrap-instance_method
In you case, you could do something like that:
Form.Element.Methods.disable=Form.Element.Methods.disable.wrap(
function(OriginalCall,ExtraParameters){
//do what you want here,
//this refer to a Form.Element.Methods object.
...
//call original method and return it's value if needed.
return OriginalCall(this);
}
);
The OriginalCall is the original prototype function you overide. You
can (or of course not) call it.
hope that help.
--
david
On 25 nov, 22:30, Hippyjim Starbrook <hippyjim.starbr...@gmail.com>
wrote:
> Hi All
>
> I'm attempting to extend the Form.Element.disable() method, so that it
> adds a "disabled" class to any element is disables.
>
> I'm attempting to replace the existing prototype function with one of
> my own, using the following code in a javascript file loaded after
> prototype.js:
>
> Form.Element.Methods.disable = function(element) {
> element = $(element);
> element.blur();
> element.disabled = true;
> element.addClassName("disabled");
> return element;
>
> }
>
> But it doesn't seem to replace the original prototype method.
>
> Am I missing something?
--
You received this message because you are subscribed to the Google Groups
"Prototype & script.aculo.us" group.
To post to this group, send email to prototype-scriptacul...@googlegroups.com.
To unsubscribe from this group, send email to
prototype-scriptaculous+unsubscr...@googlegroups.com.
For more options, visit this group at
http://groups.google.com/group/prototype-scriptaculous?hl=en.
