[piccolo2d-dev] Re: Issue 236 in piccolo2d: Add zoom event handler that responds to mouse wheel events
Updates: Status: Fixed Comment #7 on issue 236 by heue...@gmail.com: Add zoom event handler that responds to mouse wheel events http://code.google.com/p/piccolo2d/issues/detail?id=236 Turns out another zoom mode was what I was actually after, zoom about canvas center. I added that and made it the default. There is also a new MouseWheelZoomExample that demonstrates the different modes. $ svn commit -m Issue 236 ; adding zoom event handler that responds to mouse wheel events core/ examples/ Adding core/src/main/java/org/piccolo2d/event/PMouseWheelZoomEventHandler.java Adding core/src/test/java/org/piccolo2d/event/PMouseWheelZoomEventHandlerTest.java Sending examples/src/main/java/org/piccolo2d/examples/ExampleRunner.java Adding examples/src/main/java/org/piccolo2d/examples/MouseWheelZoomExample.java Transmitting file data Committed revision 1163. -- Piccolo2D Developers Group: http://groups.google.com/group/piccolo2d-dev?hl=en
[piccolo2d-dev] Re: Issue 236 in piccolo2d: Add zoom event handler that responds to mouse wheel events
Comment #6 on issue 236 by atdi...@gmail.com: Add zoom event handler that responds to mouse wheel events http://code.google.com/p/piccolo2d/issues/detail?id=236 I like it. -- Piccolo2D Developers Group: http://groups.google.com/group/piccolo2d-dev?hl=en
[piccolo2d-dev] Re: Issue 236 in piccolo2d: Add zoom event handler that responds to mouse wheel events
Comment #5 on issue 236 by heue...@gmail.com: Add zoom event handler that responds to mouse wheel events http://code.google.com/p/piccolo2d/issues/detail?id=236 Input event handlers don't have property change support, so I just went with a ZoomMode enum instead. Let me know what you think; updated patch attached. Attachments: zoom-mode-patch.txt 10.2 KB -- Piccolo2D Developers Group: http://groups.google.com/group/piccolo2d-dev?hl=en
[piccolo2d-dev] Re: Issue 236 in piccolo2d: Add zoom event handler that responds to mouse wheel events
Comment #4 on issue 236 by atdi...@gmail.com: Add zoom event handler that responds to mouse wheel events http://code.google.com/p/piccolo2d/issues/detail?id=236 Yep, I like making it configurable with a boolean option. -- Piccolo2D Developers Group: http://groups.google.com/group/piccolo2d-dev?hl=en
[piccolo2d-dev] Re: Issue 236 in piccolo2d: Add zoom event handler that responds to mouse wheel events
Comment #3 on issue 236 by heue...@gmail.com: Add zoom event handler that responds to mouse wheel events http://code.google.com/p/piccolo2d/issues/detail?id=236 Thank you for the comments. I was thinking of adding the viewZoomPoint feature, but I don't actually like that behavior. Would you be ok with making that configurable with a boolean bound property? The same might be added to PZoomEventHandler while I'm at it. And one minor fix to the patch above, -PMouseWheelZoomEventHandler() { +public PMouseWheelZoomEventHandler() { -- Piccolo2D Developers Group: http://groups.google.com/group/piccolo2d-dev?hl=en
[piccolo2d-dev] Re: Issue 236 in piccolo2d: Add zoom event handler that responds to mouse wheel events
Updates: Status: Started Owner: heue...@gmail.com Labels: Type-Enhancement Priority-Medium Effort-Low OpSys-All Toolkit-Piccolo2D.Java Component-Core Milestone-2.0 Comment #1 on issue 236 by heue...@gmail.com: Add zoom event handler that responds to mouse wheel events http://code.google.com/p/piccolo2d/issues/detail?id=236 Patch to svn trunk attached. Attachments: patch.txt 7.0 KB -- Piccolo2D Developers Group: http://groups.google.com/group/piccolo2d-dev?hl=en