[piccolo2d-dev] Issue 106 in piccolo2d: toImage array is too simple in its fill behaviour
Updates: Status: Fixed Comment #7 on issue 106 by allain.lalonde: toImage array is too simple in its fill behaviour http://code.google.com/p/piccolo2d/issues/detail?id=106 (No comment was entered for this change.) -- You received this message because you are listed in the owner or CC fields of this issue, or because you starred this issue. You may adjust your issue notification preferences at: http://code.google.com/hosting/settings --~--~-~--~~~---~--~~ Piccolo2D Developers Group: http://groups.google.com/group/piccolo2d-dev?hl=en -~--~~~~--~~--~--~---
[piccolo2d-dev] Issue 106 in piccolo2d: toImage array is too simple in its fill behaviour
Comment #4 on issue 106 by heuermh: toImage array is too simple in its fill behaviour http://code.google.com/p/piccolo2d/issues/detail?id=106 Copying my comment from Issue 88: I was not able to validate this fix. I've added an example that demonstrates the different fill strategies, and all the aspect cover images have no foreground. The aspect fill and exact fill images look correct to me. $ svn commit -m Issue 88 ; adding example that demonstrates the different toImage fill strategies, uncovered a possible bug with aspect cover fill strategy Adding examples/src/main/java/edu/umd/cs/piccolo/examples/ToImageExample.java Transmitting file data ... Committed revision 784. -- You received this message because you are listed in the owner or CC fields of this issue, or because you starred this issue. You may adjust your issue notification preferences at: http://code.google.com/hosting/settings --~--~-~--~~~---~--~~ Piccolo2D Developers Group: http://groups.google.com/group/piccolo2d-dev?hl=en -~--~~~~--~~--~--~---
[piccolo2d-dev] Issue 106 in piccolo2d: toImage array is too simple in its fill behaviour
Comment #1 on issue 106 by allain.lalonde: toImage array is too simple in its fill behaviour http://code.google.com/p/piccolo2d/issues/detail?id=106 Actually #2 (the first one) is the one I'll go with as the default. -- You received this message because you are listed in the owner or CC fields of this issue, or because you starred this issue. You may adjust your issue notification preferences at: http://code.google.com/hosting/settings --~--~-~--~~~---~--~~ Piccolo2D Developers Group: http://groups.google.com/group/piccolo2d-dev?hl=en -~--~~~~--~~--~--~---
[piccolo2d-dev] Issue 106 in piccolo2d: toImage array is too simple in its fill behaviour
Status: Accepted Owner: allain.lalonde Labels: Type-Enhancement Priority-Medium Effort-Low OpSys-All Toolkit-Piccolo2D.Java Component-Core Milestone-1.3 Usability New issue 106 by allain.lalonde: toImage array is too simple in its fill behaviour http://code.google.com/p/piccolo2d/issues/detail?id=106 Marcus noted the following: There are various sensible ways to expand one rectangle to fit into another. 1) it can be just strechted in both directions 2) AspectFit - it can be streched to fit into but preserve aspect ratio 2) AspectFill - it can be streched to completely fill but preserve aspect ratio Should be an input parameter. I agree and so will add another toImage method that accepts a fill strategy. The default will be #1 from the list since I think it's the most logical. -- You received this message because you are listed in the owner or CC fields of this issue, or because you starred this issue. You may adjust your issue notification preferences at: http://code.google.com/hosting/settings --~--~-~--~~~---~--~~ Piccolo2D Developers Group: http://groups.google.com/group/piccolo2d-dev?hl=en -~--~~~~--~~--~--~---