[kupu-dev] kupu updatestate patch

Andy Altepeter aaltepet at altepeter.net
Mon May 11 18:02:14 CEST 2009


Greetings,

I'm a Silva developer, working to push Silva's kupu implementation to the next 
level.  I've identified a bug in kupueditor.js which causes clicks in the 
content to trigger two state updated.  There updatestate handler is being 
called twice, because it is registered for both mouseup and click.

This causes a few problems.  One of them is mostly visual: screen flickers can 
happen, if the state of the tools changes.  The second one is that if 
updatestate triggers any AJAX requests, they happen twice.

I've attached a patch for this, and would like to see it put in the trunk.  
However, silva has it's own branch of kupu, for reasons that aren't quite 
clear to me, and this will also need to be applied there as well.

Please advise on how to proceed with this patch.

Cheers,
Andy
-------------- next part --------------
A non-text attachment was scrubbed...
Name: kupu_double-event_fix.diff
Type: text/x-diff
Size: 594 bytes
Desc: not available
Url : http://codespeak.net/pipermail/kupu-dev/attachments/20090511/e3abc137/attachment.bin 


More information about the kupu-dev mailing list