[piccolo2d-dev] Re: Issue 150 in piccolo2d: Prepare SwingLayoutNode for distribution
Comment #1 on issue 150 by samrreid: Prepare SwingLayoutNode for distribution http://code.google.com/p/piccolo2d/issues/detail?id=150 On Dec 1, Michael Heuer wrote: I think this is cool, and a good fit for piccolox.swing in extras. Could you create an issue for this? With the proper license header, unit tests, checkstyle fixes, and refactoring of the example into a separate class, this could go into 1.3. michael -- 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] Re: Issue 150 in piccolo2d: Prepare SwingLayoutNode for distribution
Comment #2 on issue 150 by samrreid: Prepare SwingLayoutNode for distribution http://code.google.com/p/piccolo2d/issues/detail?id=150 See this thread for context: http://groups.google.com/group/piccolo2d-users/browse_thread/thread/741bca3c0aa2d96e?pli=1 -- 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] Re: Issue 150 in piccolo2d: Prepare SwingLayoutNode for distribution
Comment #3 on issue 150 by samrreid: Prepare SwingLayoutNode for distribution http://code.google.com/p/piccolo2d/issues/detail?id=150 Chris suggested that we also spend some additional time to verify that (1) we're not exposing anything that we'll regret later, and (2) method names are logical. I agree that we need to make sure the public API is acceptable before publishing this. -- 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] Re: Issue 150 in piccolo2d: Prepare SwingLayoutNode for distribution
Comment #4 on issue 150 by samrreid: Prepare SwingLayoutNode for distribution http://code.google.com/p/piccolo2d/issues/detail?id=150 I don't have plans for working on this in the near future, if someone wants to work on it. Here's a quick itemized summary of proposed work: 1. Add to SVN in piccolox 2. Add license header 3. Add unit tests 4. Add checkstyle fixes 5. Refactor sample usage to separate class 6. Make sure public interface is acceptable 7. Make sure private internal method naming, etc is acceptable -- 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] Re: Issue 150 in piccolo2d: Prepare SwingLayoutNode for distribution
Updates: Status: Accepted Owner: allain.lalonde Labels: Type-Enhancement Priority-Low Effort-Low OpSys-All Toolkit-Piccolo2D.Java Component-Extras Milestone-1.3 Usability Comment #5 on issue 150 by allain.lalonde: Prepare SwingLayoutNode for distribution http://code.google.com/p/piccolo2d/issues/detail?id=150 I'll do it. -- 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] Re: Issue 150 in piccolo2d: Prepare SwingLayoutNode for distribution
Comment #6 on issue 150 by allain.lalonde: Prepare SwingLayoutNode for distribution http://code.google.com/p/piccolo2d/issues/detail?id=150 I'd tried something like this in the past and failed, so I'm very much liking this code. -- 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] Re: Issue 150 in piccolo2d: Prepare SwingLayoutNode for distribution
Comment #7 on issue 150 by allain.lalonde: Prepare SwingLayoutNode for distribution http://code.google.com/p/piccolo2d/issues/detail?id=150 I'm looking at the Anchor approach and am wondering if it's too restrictive. It's basically a fill strategy for whatever bounds the LayoutManager assigns to a node. It would seem that changing the bounds of the nodes is as valid an approach as pinning the node to the WEST of the box but the current naming convention doesn't make it clear that Anchor.FILL might be a valid fill strategy. Thoughts? -- 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] Re: Issue 150 in piccolo2d: Prepare SwingLayoutNode for distribution
Comment #8 on issue 150 by heue...@gmail.com: Prepare SwingLayoutNode for distribution http://code.google.com/p/piccolo2d/issues/detail?id=150 I would vote to leave as-is for now. Would you want to mark this one as Fixed, Allain? -- 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] Re: Issue 150 in piccolo2d: Prepare SwingLayoutNode for distribution
Updates: Status: Fixed Comment #9 on issue 150 by allain.lalonde: Prepare SwingLayoutNode for distribution http://code.google.com/p/piccolo2d/issues/detail?id=150 (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] Re: Issue 150 in piccolo2d: Prepare SwingLayoutNode for distribution
Updates: Status: Verified Comment #10 on issue 150 by heue...@gmail.com: Prepare SwingLayoutNode for distribution http://code.google.com/p/piccolo2d/issues/detail?id=150 (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