popups

Roman Kennke roman at kennke.org
Tue Apr 3 03:16:46 PDT 2012


Hi Glen,

> FEST uses right click, i.e. in BasicRobot.showPopupMenu
> 
>     click(invoker, location, RIGHT_BUTTON, 1);
> 
> So I would vote to just use right click for now.
> 
> I was playing around with the source and hacked in what I think is right for setting the popup value in the MouseEvent.  We noticed a problem in CTCRobotPeer.mouseRelease where it was using the wrong values to set currentModifiers and also moved where it was setting that value to after the event was sent.
> 
> Attached is the patch file along with some tests.

Sorry for the delay. The change looks good. I wanted to push your change
some days ago already, but apparently something got broken in OpenJDK's
repository setup, I currently don't have push rights. As soon as that is
fixed, I will push your changes (along with some minor fixes I have made
to the project setup).

Following that, I will make a release and upload Cacio to the
Sonatype/Maven Central repository so that it can actually be used from
Maven.

Cheers,
Roman





More information about the caciocavallo-dev mailing list