ICEfaces
  1. ICEfaces
  2. ICE-751

New selectOneRadio + selectManyCheckbox grouping mechanism

    Details

    • Type: New Feature New Feature
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.5
    • Fix Version/s: 1.7DR#3, 1.7
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      Operating System: All
      Platform: PC

      Description

      We need a new mechanism to support the ability for developers to define a
      logical group for selectOneRadio components.

      selectOneRadio components that are assigned to the same logical group will
      affect each others state as follows:

        1. Only one selectOneRadio component can be selected at one time in a logical
      group.

        2. Selecting a selectOneRadio component will automatically deselect any
      previously selected selectOneRadio component in the same logical group.

      This mechanism will make it possible to have multiple logical groupings of
      selecOneRadio components on the same page, whereas today all selectOneRadio
      components interact with each other on the same page, which is very limiting.

      This concept is similar to the logical grouping of valid target
      panelPositionable components so we should ensure that the mechanism used to
      define the member groups in both cases is consistent.

        Activity

        Ken Fyten created issue -
        Hide
        Adnan Durrani added a comment -

        This feature has been moved to the next release, I discussed this issue with
        the Ken and Philip. Ken suggested that we should implement this feature for
        both selectOneRadio and selectManyCheckbox all together and should follow the
        same attributes and terminology as Myfaces did for their checkbox component.

        Please see the code snippet of myfaces' checkbox implementation:
        <x:selectManyCheckbox id="myCheckbox" layout="spread" binding="#

        {textBean.bindItem1}

        " >
        <f:selectItems value="#

        {textBean.items2}

        "/>
        </x:selectManyCheckbox>

        <h:dataTable var="data4" value="#

        {textBean.dataModel2}

        ">
        <h:column>
        <x:checkbox for="myCheckbox" index="#

        {textBean.dataModel2.rowIndex}

        "/>
        </h:column>
        </h:dataTable>

        Show
        Adnan Durrani added a comment - This feature has been moved to the next release, I discussed this issue with the Ken and Philip. Ken suggested that we should implement this feature for both selectOneRadio and selectManyCheckbox all together and should follow the same attributes and terminology as Myfaces did for their checkbox component. Please see the code snippet of myfaces' checkbox implementation: <x:selectManyCheckbox id="myCheckbox" layout="spread" binding="# {textBean.bindItem1} " > <f:selectItems value="# {textBean.items2} "/> </x:selectManyCheckbox> <h:dataTable var="data4" value="# {textBean.dataModel2} "> <h:column> <x:checkbox for="myCheckbox" index="# {textBean.dataModel2.rowIndex} "/> </h:column> </h:dataTable>
        Icefaces Administrator made changes -
        Field Original Value New Value
        issue.field.bugzillaimportkey 782 12020
        Ken Fyten made changes -
        Affects Version/s 1.5 [ 10027 ]
        Affects Version/s 1.0.1 [ 10025 ]
        Ken Fyten made changes -
        Fix Version/s 2.0 [ 10032 ]
        Ken Fyten made changes -
        Assignee Adnan Durrani [ adnan.durrani ]
        Philip Breau made changes -
        Michael Thiem made changes -
        Attachment Test_MultipleRadioButtons.war [ 10673 ]
        Michael Thiem made changes -
        Attachment Test_MultipleRadioButtons.rar [ 10674 ]
        Ken Fyten made changes -
        Fix Version/s 1.7 [ 10080 ]
        Assignee Adnan Durrani [ adnan.durrani ]
        Ken Fyten made changes -
        Fix Version/s 1.7DR#3 [ 10112 ]
        Fix Version/s 1.7 [ 10080 ]
        Ken Fyten made changes -
        Summary New selectOneRadio grouping mechanism New selectOneRadio + selectManyCheckbox grouping mechanism
        Ken Fyten made changes -
        Assignee Adnan Durrani [ adnan.durrani ] Yip Ng [ yip.ng ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #15164 Tue Nov 13 13:34:00 MST 2007 yip.ng ICE-751
        Added implementation of radio buttons with a "spread" layout as in MyFaces.
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/renderkit/dom_html_basic/SelectManyCheckboxListRenderer.java
        Commit graph ADD /icefaces/trunk/icefaces/component-metadata/src/main/resources/conf/ice_properties/ice-radio-props.xml
        Commit graph ADD /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/ext/HtmlRadio.java
        Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/renderkit/dom_html_basic/RadioRenderer.java
        Commit graph MODIFY /icefaces/trunk/icefaces/component-metadata/src/main/resources/conf/extended-faces-config.xml
        Commit graph ADD /icefaces/trunk/icefaces/component-metadata/src/main/resources/conf/components/ice-radio-component.xml
        Commit graph MODIFY /icefaces/trunk/icefaces/component/conf/META-INF/faces-config.xml
        Commit graph ADD /icefaces/trunk/icefaces/component-metadata/src/main/resources/conf/components/ice-radio-renderer.xml
        Hide
        yip.ng added a comment -

        Screenshot showing separate group of radio buttons (Select destination). The implementation is similar to that of MyFaces:

        <ice:selectOneRadio id="SlctDest" layout="spread".../>
        ...
        <ice:radio for="SlctDest" index="0"/>
        ...
        <ice:radio for="SlctDest" index="3"/>

        Show
        yip.ng added a comment - Screenshot showing separate group of radio buttons (Select destination). The implementation is similar to that of MyFaces: <ice:selectOneRadio id="SlctDest" layout="spread".../> ... <ice:radio for="SlctDest" index="0"/> ... <ice:radio for="SlctDest" index="3"/>
        yip.ng made changes -
        Attachment screenshot-1.jpg [ 10730 ]
        Ken Fyten made changes -
        Priority Minor [ 4 ] Major [ 3 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #15171 Tue Nov 13 17:39:36 MST 2007 yip.ng ICE-751
        Added implementation of checkboxes with a "spread" layout as in MyFaces.
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/renderkit/dom_html_basic/SelectManyCheckboxListRenderer.java
        Commit graph ADD /icefaces/trunk/icefaces/component-metadata/src/main/resources/conf/ice_properties/ice-checkbox-props.xml
        Commit graph ADD /icefaces/trunk/icefaces/component-metadata/src/main/resources/conf/components/ice-checkbox-renderer.xml
        Commit graph ADD /icefaces/trunk/icefaces/component-metadata/src/main/resources/conf/components/ice-checkbox-component.xml
        Commit graph MODIFY /icefaces/trunk/icefaces/component-metadata/src/main/resources/conf/extended-faces-config.xml
        Commit graph ADD /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/ext/HtmlCheckbox.java
        Commit graph MODIFY /icefaces/trunk/icefaces/component/conf/META-INF/faces-config.xml
        Hide
        yip.ng added a comment -

        Screenshot showing separate group of checkboxes (Select foods). The implementation is similar to that of MyFaces:

        <ice:selectManyCheckbox id="SlctFoods" layout="spread".../>
        ...
        <ice:checkbox for="SlctFoods" index="0"/>
        ...
        <ice:checkbox for="SlctFoods" index="3"/>

        Show
        yip.ng added a comment - Screenshot showing separate group of checkboxes (Select foods). The implementation is similar to that of MyFaces: <ice:selectManyCheckbox id="SlctFoods" layout="spread".../> ... <ice:checkbox for="SlctFoods" index="0"/> ... <ice:checkbox for="SlctFoods" index="3"/>
        yip.ng made changes -
        Attachment screenshot-2.jpg [ 10731 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #15185 Wed Nov 14 14:07:44 MST 2007 yip.ng ICE-751
        Updated documentation.
        Added more error checking.
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/renderkit/dom_html_basic/SelectManyCheckboxListRenderer.java
        Commit graph MODIFY /icefaces/trunk/icefaces/component-metadata/src/main/resources/conf/ice_properties/ice-checkbox-props.xml
        Commit graph MODIFY /icefaces/trunk/icefaces/component-metadata/src/main/resources/conf/components/ice-checkbox-renderer.xml
        Commit graph MODIFY /icefaces/trunk/icefaces/component-metadata/src/main/resources/conf/ice_properties/ice-radio-props.xml
        Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/ext/HtmlRadio.java
        Commit graph MODIFY /icefaces/trunk/icefaces/component-metadata/src/main/resources/conf/components/ice-checkbox-component.xml
        Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/renderkit/dom_html_basic/RadioRenderer.java
        Commit graph MODIFY /icefaces/trunk/icefaces/component-metadata/src/main/resources/conf/components/ice-radio-component.xml
        Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/ext/HtmlCheckbox.java
        Commit graph MODIFY /icefaces/trunk/icefaces/component-metadata/src/main/resources/conf/components/ice-radio-renderer.xml
        yip.ng made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Issue Type Improvement [ 4 ] New Feature [ 2 ]
        Affects [Documentation (User Guide, Ref. Guide, etc.), Sample App./Tutorial]
        Ken Fyten made changes -
        Fix Version/s 1.7 [ 10080 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Assignee Yip Ng [ yip.ng ]

          People

          • Assignee:
            Unassigned
            Reporter:
            Ken Fyten
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: