Comment 2 for bug 1092651

Revision history for this message
Sam Spilsbury (smspillaz) wrote :

Hi Robert.

Thanks for the patch. I've uploaded a new branch and proposed it for merging with your patch included, and some coding style things cleaned up. I also removed the check for mouseButtonModifier = 0 - as that is a non-error case.

It passes tests, but I do not have the time to manual-test it. Can you check if it remains true to your original patch?