groovy
  1. groovy
  2. GROOVY-501

SwingBuilder - add more components to support ButtonDemo

    Details

    • Type: Improvement Improvement
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.0-beta-6
    • Component/s: None
    • Labels:
      None
    • Number of attachments :
      4

      Description

      For a test of the relative size and ease of a Groovy swing applicaiton I though of re-implementing the SwingSet from the JFC demos in groovy. It wound up being about a third of the size of the original!

      To do a really groovy job I needed to add some more items and concepts to the SwingBuilder.jar.

      notable changes:

      • add more methiods from javax.swing.Box for glue, struts, and rigid areas
      • add buttonGroup case for AbstractButtons so we don't have to dive into the model to set a button group on a checkBox or radioButton

      tweaks:

      • add support for adding components to a tabbed pane explicitly
      • introduce some "pass through nodes" where the widget is not nesscicarily created but provided by the user (an elaboration of the widget() node from GROOVY-333) they are widget, action, and tableModel.
      • refactor code where attribures are set as java beans proeprties on the widget.

      This patch may contain stuff from the GROOVY-333 patch. I don't thuink it's been applied yet.

      Also, ButtonDemo.groovy is the portion of the Buttons tab from SwingSet in ButtonDemo.java, sans the images.

      1. ButtonDemo.groovy
        8 kB
        Danno Ferrin
      2. SwingBuilder.java
        28 kB
        Danno Ferrin
      3. swingbuilderboxy-patch.txt
        13 kB
        Danno Ferrin
      4. swingbuilderboxy-patch-unix.txt
        13 kB
        Danno Ferrin

        Activity

        Hide
        Danno Ferrin added a comment -

        strange, the patch file in cygwin liked it just fine. is it \r\n vs. \n? If so I did unix lin endingx in the -unix patch, otherwise I attached the entire file as well.

        Show
        Danno Ferrin added a comment - strange, the patch file in cygwin liked it just fine. is it \r\n vs. \n? If so I did unix lin endingx in the -unix patch, otherwise I attached the entire file as well.
        Hide
        Guillaume Laforge added a comment -

        Patch applied, thank you Danno.
        Could you also have a look back at GROOVY-333, to see if some of its enhancements could be added ?

        Show
        Guillaume Laforge added a comment - Patch applied, thank you Danno. Could you also have a look back at GROOVY-333 , to see if some of its enhancements could be added ?

          People

          • Assignee:
            Guillaume Laforge
            Reporter:
            Danno Ferrin
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: