ICEfaces
  1. ICEfaces
  2. ICE-4142

Styles are not applied to ice:radio

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.8RC1
    • Fix Version/s: 1.8RC2, 1.8
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      icefaces rev18425

      Description

      Following scenario :

      <ice:selectOneRadio id="woi13" layout="spread" partialSubmit="false"
      value="#{...}" disabled="#{...}" disabledClass="rbHideText" enabledClass="rbHideText">
         <f:selectItems value="#{list.value}" />
      </ice:selectOneRadio>

      <ice:dataTable value="#{lista.value}" ...
         <ice:column>
             <ice:radio for="woi13" index="#{element.value}" />
        </ice:column>
      <ice:dataTable>

      Classes styleClass, disabledClass, enabledClass are not added to output.

      BTW
      If style patch is imposiible then new feature will be usable.
      Spread radio's are generated with label's. In table those labels are not necessary because users got columns.
      Ice:radio with new attribute for disabling labels can help users.

        Activity

        Hide
        yip.ng added a comment -

        Readonly attribute is working now. See attached screenshot. Fixed by changing the call to the PassThruAttributeRenderer. No need for the patch.

        Show
        yip.ng added a comment - Readonly attribute is working now. See attached screenshot. Fixed by changing the call to the PassThruAttributeRenderer. No need for the patch.
        Hide
        Krashan Brahmanjara added a comment -

        "We don't use the "enabledClass" and "disabledClass"."

        Current revision 18478. In tld users still see "enabledClass" and "disabledClass".

        Show
        Krashan Brahmanjara added a comment - "We don't use the "enabledClass" and "disabledClass"." Current revision 18478. In tld users still see "enabledClass" and "disabledClass".
        Hide
        yip.ng added a comment -

        New JIRA created: ICE-4187.

        Show
        yip.ng added a comment - New JIRA created: ICE-4187 .
        Hide
        Krashan Brahmanjara added a comment -

        I see a label generation bug after last changes .

        The same scenario described at beggining. Now radio's are generated with correct value and incorrect labels.
        Example for one item (value=32 but label is from third item)

        <input id="UE:icePnlTbSet:0:woi13:_4" type="radio" value="32" onkeypress="Ice.util.radioCheckboxEnter(form,this,event);" onfocus="setFocus(this.id);" onblur="setFocus('');" name="UE:icePnlTbSet:0:woi13" disabled="disabled"/>
        <label class="iceSelOneRb-dis" for="UE:icePnlTbSet:0:woi13:_4">DEM</label>

        BTW
        Styles are now applied but only to label element, not to input.

        Show
        Krashan Brahmanjara added a comment - I see a label generation bug after last changes . The same scenario described at beggining. Now radio's are generated with correct value and incorrect labels. Example for one item (value=32 but label is from third item) <input id="UE:icePnlTbSet:0:woi13:_4" type="radio" value="32" onkeypress="Ice.util.radioCheckboxEnter(form,this,event);" onfocus="setFocus(this.id);" onblur="setFocus('');" name="UE:icePnlTbSet:0:woi13" disabled="disabled"/> <label class="iceSelOneRb-dis" for="UE:icePnlTbSet:0:woi13:_4">DEM</label> BTW Styles are now applied but only to label element, not to input.
        Hide
        yip.ng added a comment -

        Value is a key, not an index. It could be a string, not a number. It is matched to the select item with the same value, not the same index. The index is used solely for layout purposes.

        For applying the style to input, see ICE-4187.

        Show
        yip.ng added a comment - Value is a key, not an index. It could be a string, not a number. It is matched to the select item with the same value, not the same index. The index is used solely for layout purposes. For applying the style to input, see ICE-4187 .

          People

          • Assignee:
            Unassigned
            Reporter:
            Krashan Brahmanjara
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: