Details

    • Type: New Feature New Feature
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.1
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      ICEfaces 4
    • Affects:
      Documentation (User Guide, Ref. Guide, etc.), Sample App./Tutorial

      Description

      This JIRA is to add a new ace:radioButtons component that would support stamping out a set of radioButtons based on one component tag, similar to the h:selectOneRadioButton component.

      This would support all the styling/attributes of the existing ace:radioButton component, but would support f:selectItems to define the buttons and would use a single value attribute to define the currently selected radioButton.

        Activity

        Hide
        Ken Fyten added a comment - - edited

        Note that this feature was previously supported in the ice:selectOneRadio component.

        <ice:selectOneRadio id="SlctDrk"
            value="#{selectionTags.selectedDrink}"
            valueChangeListener="#{selectionTags.effectChangeListener}"
            partialSubmit="true" >
            <f:selectItems  id="SlctDrkItms" value="#{selectionTags.drinkItems}"/>
        </ice:selectOneRadio>
        
        Show
        Ken Fyten added a comment - - edited Note that this feature was previously supported in the ice:selectOneRadio component. <ice:selectOneRadio id= "SlctDrk" value= "#{selectionTags.selectedDrink}" valueChangeListener= "#{selectionTags.effectChangeListener}" partialSubmit= " true " > <f:selectItems id= "SlctDrkItms" value= "#{selectionTags.drinkItems}" /> </ice:selectOneRadio>
        Hide
        Arturo Zambrano added a comment -

        r45855: committed new ace:radioButtons component

        Show
        Arturo Zambrano added a comment - r45855: committed new ace:radioButtons component
        Hide
        Arturo Zambrano added a comment -

        r45861: committed some styling adjustments
        r45862: added label, indicator, and required styling demos

        Show
        Arturo Zambrano added a comment - r45861: committed some styling adjustments r45862: added label, indicator, and required styling demos
        Hide
        Arturo Zambrano added a comment -

        r45874: modified rendering behaviour when the submitted value is invalid, to render the invalid submitted value instead of the previous valid value, in order to apply invalid styling consistently and allow the user to correct the input; corrected radioButtons behaviour so that the radio buttons don't behave like checkboxes (i.e. one selected they can't be deselected by clicking on them again)

        Show
        Arturo Zambrano added a comment - r45874: modified rendering behaviour when the submitted value is invalid, to render the invalid submitted value instead of the previous valid value, in order to apply invalid styling consistently and allow the user to correct the input; corrected radioButtons behaviour so that the radio buttons don't behave like checkboxes (i.e. one selected they can't be deselected by clicking on them again)
        Hide
        Arturo Zambrano added a comment -

        r45875: removed unnecessary resource files from the resource examples; modified the way the values are displayed to make them more presentable.

        Show
        Arturo Zambrano added a comment - r45875: removed unnecessary resource files from the resource examples; modified the way the values are displayed to make them more presentable.
        Hide
        Carlo Guglielmin added a comment - - edited

        Re-opening because if "labelPosition" is not specified there is no text shown for each radio button (see screenshot). This is unrelated to any "label" attribute usage.
        This prevents ace:radioButtons from being a direct drop in replacement for h:selectOneRadio.

        Ideally the text just displays properly, or worst case labelPosition should default to the traditional "left".

        Show
        Carlo Guglielmin added a comment - - edited Re-opening because if "labelPosition" is not specified there is no text shown for each radio button (see screenshot). This is unrelated to any "label" attribute usage. This prevents ace:radioButtons from being a direct drop in replacement for h:selectOneRadio. Ideally the text just displays properly, or worst case labelPosition should default to the traditional "left".
        Hide
        Carlo Guglielmin added a comment - - edited

        Screenshot demonstrating the missing labels unless "labelPosition" is manually specified.

        Show
        Carlo Guglielmin added a comment - - edited Screenshot demonstrating the missing labels unless "labelPosition" is manually specified.
        Hide
        Arturo Zambrano added a comment -

        r46527: changes in label position to make ace:radioButtons behave like h:selectOneRadio : 1) The default value for the 'labelPosition' attribute is now set to 'right' instead of 'none', 2) If the 'labelPosition' attribute is set to null or to the empty string, the label is still displayed to the right of the radio button; only when 'labelPosition' is set explicitly to 'none' the label won't be displayed

        Show
        Arturo Zambrano added a comment - r46527: changes in label position to make ace:radioButtons behave like h:selectOneRadio : 1) The default value for the 'labelPosition' attribute is now set to 'right' instead of 'none', 2) If the 'labelPosition' attribute is set to null or to the empty string, the label is still displayed to the right of the radio button; only when 'labelPosition' is set explicitly to 'none' the label won't be displayed
        Hide
        Carlo Guglielmin added a comment -

        Looks good from my Emporium testing.

        Show
        Carlo Guglielmin added a comment - Looks good from my Emporium testing.
        Hide
        Arturo Zambrano added a comment -

        r46531: removed 'inField' value from 'labelPosition' TLD documentation.

        Thank you for observing that, Carmen. The 'inField' value has now been removed from the ace:radioButtons documentation, since ace:radioButton doesn't support it either.

        Show
        Arturo Zambrano added a comment - r46531: removed 'inField' value from 'labelPosition' TLD documentation. Thank you for observing that, Carmen. The 'inField' value has now been removed from the ace:radioButtons documentation, since ace:radioButton doesn't support it either.

          People

          • Assignee:
            Arturo Zambrano
            Reporter:
            Ken Fyten
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: