Drawing tears when dragging or resizing path nodes

GoogleCodeArchive / piccolo2d

Automatically exported from code.google.com/p/piccolo2d

0 stars 0 forks source link

Drawing tears when dragging or resizing path nodes #180

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago

Drawing tears when dragging or resizing path nodes in e.g. SelectionExample or 
TwoCanvasExample.

This happens for me with OpenJDK 1.6 on linux and JDK 1.5, 1.6, & 1.7 on 
windows, with svn trunk revision r1025.  It does not happen with the 1.3 src 
release or svn branch release-1.3.

See attached screenshots.

Original issue reported on code.google.com by heue...@gmail.com on 14 Jun 2010 at 8:32

Attachments:

GoogleCodeExporter commented 9 years ago

I've reproduced this same issue on Mac OS X/JDK 1.6.

Original comment by atdi...@gmail.com on 15 Jun 2010 at 12:28

GoogleCodeExporter commented 9 years ago

Issue 184 has been merged into this issue.

Original comment by heue...@gmail.com on 2 Aug 2010 at 2:22

GoogleCodeExporter commented 9 years ago

I'm not 100% sure that the problem in Issue 184 is the same as the one in this 
ticket.  For example, the problem indicated in Issue 184's screenshot suggests 
a bounding problem caused by Piccolo not accounting for antialiasing in 
computing PNode bounds:
http://lionel.victor.free.fr/Capture-OffsetVsTranslateExample.png

Original comment by samrr...@gmail.com on 2 Aug 2010 at 4:59

GoogleCodeExporter commented 9 years ago

Does this bit of cleanup on trunk in PNode.validateFullBounds() look ok?

revision page

http://code.google.com/p/piccolo2d/source/detail?spec=svn1008&r=1008

diff to previous

http://code.google.com/p/piccolo2d/source/diff?path=/piccolo2d.java/trunk/core/s
rc/main/java/edu/umd/cs/piccolo/PNode.java&format=side&r=1008&old_path=/piccolo2
d.java/trunk/core/src/main/java/edu/umd/cs/piccolo/PNode.java&old=1006

Original comment by heue...@gmail.com on 2 Aug 2010 at 9:45

GoogleCodeExporter commented 9 years ago

Reverting PNode.validateFullBounds() to r1006 on trunk clears up the problems 
in the examples described above.

Should I commit the reversion or try to rewrite the cleanup in r1008?

Original comment by heue...@gmail.com on 2 Aug 2010 at 10:02

Changed state: Accepted
Added labels: Priority-Critical, Effort-Medium, Component-Core, Milestone-2.0, Usability

GoogleCodeExporter commented 9 years ago

$ svn commit -m "Issue 180 ; rolling back r1008 to r1006" .
Sending        core/src/main/java/org/piccolo2d/PNode.java
Transmitting file data .
Committed revision 1040.

Original comment by heue...@gmail.com on 6 Aug 2010 at 4:03

Changed state: Fixed

GoogleCodeExporter commented 9 years ago

Issue 184 has been merged into this issue.

Original comment by heue...@gmail.com on 24 Aug 2010 at 2:46

GoogleCodeExporter commented 9 years ago

Original comment by heue...@gmail.com on 26 Nov 2013 at 8:47

Changed state: Verified