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/

Reply via email to