Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 5.3
    • Fix Version/s: 5.3
    • Component/s: tapestry-core
    • Labels:
      None

      Description

      Currently it's not possible to select/unselect leaf nodes. This limitation makes the Tree component unusable as the only way to interact with is to expand nodes. A common use case for a tree is to execute some logic upon a leaf selection. We should fix it before 5.3 goes final.

        Issue Links

          Activity

          Hide
          Hudson added a comment -

          Integrated in tapestry-trunk-freestyle #411 (See https://builds.apache.org/job/tapestry-trunk-freestyle/411/)
          TAP5-1562: Introduced TreeSelectionModel.

          drobiazko : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1143969
          Files :

          • /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/tree/DefaultTreeExpansionModel.java
          • /tapestry/tapestry5/trunk/tapestry-core/src/test/groovy/org/apache/tapestry5/integration/app1/TreeTests.groovy
          • /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/corelib/components/Tree.java
          • /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/tree/TreeSelectionModel.java
          • /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/tree/TreeExpansionModel.java
          • /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/tree/DefaultTreeSelectionModel.java
          Show
          Hudson added a comment - Integrated in tapestry-trunk-freestyle #411 (See https://builds.apache.org/job/tapestry-trunk-freestyle/411/ ) TAP5-1562 : Introduced TreeSelectionModel. drobiazko : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1143969 Files : /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/tree/DefaultTreeExpansionModel.java /tapestry/tapestry5/trunk/tapestry-core/src/test/groovy/org/apache/tapestry5/integration/app1/TreeTests.groovy /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/corelib/components/Tree.java /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/tree/TreeSelectionModel.java /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/tree/TreeExpansionModel.java /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/tree/DefaultTreeSelectionModel.java
          Hide
          Hudson added a comment -

          Integrated in tapestry-trunk-freestyle #401 (See https://builds.apache.org/job/tapestry-trunk-freestyle/401/)
          TAP5-1562: Made tree leafs selectable

          drobiazko : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1142070
          Files :

          • /tapestry/tapestry5/trunk/tapestry-core/src/main/resources/org/apache/tapestry5/tree.js
          • /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/tree/DefaultTreeExpansionModel.java
          • /tapestry/tapestry5/trunk/tapestry-core/src/test/groovy/org/apache/tapestry5/integration/app1/TreeTests.groovy
          • /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/EventConstants.java
          • /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/corelib/components/Tree.java
          • /tapestry/tapestry5/trunk/tapestry-core/src/main/resources/org/apache/tapestry5/tree.css
          • /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/tree/TreeExpansionModel.java
          Show
          Hudson added a comment - Integrated in tapestry-trunk-freestyle #401 (See https://builds.apache.org/job/tapestry-trunk-freestyle/401/ ) TAP5-1562 : Made tree leafs selectable drobiazko : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1142070 Files : /tapestry/tapestry5/trunk/tapestry-core/src/main/resources/org/apache/tapestry5/tree.js /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/tree/DefaultTreeExpansionModel.java /tapestry/tapestry5/trunk/tapestry-core/src/test/groovy/org/apache/tapestry5/integration/app1/TreeTests.groovy /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/EventConstants.java /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/corelib/components/Tree.java /tapestry/tapestry5/trunk/tapestry-core/src/main/resources/org/apache/tapestry5/tree.css /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/tree/TreeExpansionModel.java
          Hide
          Igor Drobiazko added a comment -

          Actually I was thinking about selecting a leaf by clicking on it.

          Show
          Igor Drobiazko added a comment - Actually I was thinking about selecting a leaf by clicking on it.
          Hide
          Howard M. Lewis Ship added a comment -

          Selection would be nice; I've done that in a tree component for a client. You could override the block that renders the tree elements to include a checkbox, but that's a bit too roll-your-own.

          Show
          Howard M. Lewis Ship added a comment - Selection would be nice; I've done that in a tree component for a client. You could override the block that renders the tree elements to include a checkbox, but that's a bit too roll-your-own.

            People

            • Assignee:
              Igor Drobiazko
              Reporter:
              Igor Drobiazko
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development