Heya gilles!
ah, thanks for that! :) I'm actually not much of a coder, just an animator
who happens to need to pop open vi every once in a while and hack away at
stuff. :)
(btw, the name's jason).
Thanks again for a great plugin!!
-jason
On 11/29/06, [EMAIL PROTECTED] <[EMAIL PROTECTED]> wrote:
Hi Stefan,
Good one, i'll add this to the iSelectable core later on today.
There is one error in your code though:
> if (jQuery.selectedone == true && this.f.onselect) {
> this.f.onselect(jQuery.Selectserialize(jQuery.attr(this,'id')));
> } else {
> this.f.ondeselect();
> }
>
This should read:
> if (jQuery.selectedone == true && this.f.onselect) {
> this.f.onselect(jQuery.Selectserialize(jQuery.attr(this,'id')));
> } else if (this.f.ondeselect) {
> this.f.ondeselect();
> }
>
Or even this:
> if (jQuery.selectedone == true) {
> if (this.f.onselect)
> this.f.onselect(jQuery.Selectserialize(jQuery.attr(this,
'id')));
> else if (this.f.ondeselect)
> this.f.ondeselect();
> }
>
This depends on how the code is right now, but i can't see it from here.
The point is, you forgot to check if there is a "ondeselect" assigned.
-- Gilles
_______________________________________________
jQuery mailing list
[email protected]
http://jquery.com/discuss/
--
jason schleifer
ah-ni-may-tor | weirdo
http://jonhandhisdog.com/
_______________________________________________
jQuery mailing list
[email protected]
http://jquery.com/discuss/