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?
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?