GoogleCodeArchive / piccolo2d

Automatically exported from code.google.com/p/piccolo2d
0 stars 0 forks source link

PStyledTextEventHandler does not scale #71

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Run the TextExample
2. Add a text entry 
3. Zoom in and add another entry

What is the expected output? What do you see instead?
Instead of the second text entry being smaller that the first entry instead
it stays at the same scale.

What version of the product are you using? On what operating system?
Piccolo 1.2.1 Java

Original issue reported on code.google.com by norder...@gmail.com on 25 Nov 2008 at 12:23

GoogleCodeExporter commented 9 years ago

Original comment by heue...@gmail.com on 16 Mar 2009 at 5:59

GoogleCodeExporter commented 9 years ago

Original comment by heue...@gmail.com on 16 Mar 2009 at 6:03

GoogleCodeExporter commented 9 years ago

Original comment by heue...@gmail.com on 15 May 2009 at 6:42

GoogleCodeExporter commented 9 years ago
Fixed in r532; made resulting PStyledText take the camera's scale into account.
Also, made right click call stopEditing, so that the input field was always the
same size as the resulting text would be, was weird otherwise

Original comment by allain.lalonde on 21 Jul 2009 at 2:58

GoogleCodeExporter commented 9 years ago
r532 seems to fix the original issue.

Is this behavior correct?

1. Run the TextExample
2. Add a text entry "Large text"
3. Zoom in and add another entry "Small text"
4. "Large text" is large and "Small text" is small
5. Click on "Large text" to edit, and right click to stop editing
6. "Large text" is now small

Do we need to save the scale at which the PStyledText was initially created, or
should it adapt to the scale at which it was most recently edited (which is the
apparent behavior with r532)?

Original comment by heue...@gmail.com on 28 Jul 2009 at 2:13

GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
I think so. Seems more logical. I'll make the change.  re-opening.

Original comment by allain.lalonde on 28 Jul 2009 at 3:59

GoogleCodeExporter commented 9 years ago
Fixed in r652

Original comment by allain.lalonde on 5 Aug 2009 at 4:19

GoogleCodeExporter commented 9 years ago

Original comment by heue...@gmail.com on 21 Oct 2009 at 2:41