matthewlein / jQuery-jSlots

jQuery slot machine insanity! Turns any list into a slot machine (you provide the insanity)
184 stars 95 forks source link

Do not clone only the first li item #5

Closed marvink closed 11 years ago

marvink commented 11 years ago

you should clone:

$list.find('li').clone().appendTo($list);

instead of

$li.clone().appendTo($list);

otherwise the first letter has a bigger chance to win.

matthewlein commented 11 years ago

Hello,

Just looking back at this. I'm fairly sure it is accounted for in the 1-based index. Its duplicated so the animation looks good. I don't really like the 1-based index idea, but thats how I did it and I don't really have time/reason to rework it.

Thanks for looking out though.

Matt