Fixes #770

[evrard]

This update add a lock flag that will avoid multiple click() call while the effect is not over. This fix solve the angry clicks on slideshow forward/backward buttons and probably more issues.

Another way to solve multiple click() issues would be to stack click() call and execute them one after the other.

[evrard]

Another idea (my best one) about multiple click() issues would be that:

• keep the lock flag as is, so on click, lock, and on effect callback, release it
• when click() is called while it is locked, keep in mind the index to load, and apply it on effect callback

So the process should look like :

• call click( i1 ): assign lock and launch effect
• call click( i2 ): locked, keep i2 in mind
• click( i1 ) is over: release lock and call click( i2 )
• call click( i3 ): locked, keep i3 in mind
• call click( i4 ): locked, keep i4 in mind
• click( i2 ) is over: release lock and call click( i4 )

What do you think about it ?

Loops

[alibby251]

Hi evrard,

Thanks for creating this patch - any chance you could please re-base it on the current dev branch? I'm using this to form what will become Tools 1.2.8.