ICEfaces
  1. ICEfaces
  2. ICE-2170

disabled="true" not working as expected for <ice:commandLink>

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.7DR#1
    • Fix Version/s: 1.7DR#3, 1.7
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      All

      Description

      Using disabled="true" in <ice:commandLink> doesn't disable the link in any way. It renders a 'disabled' attribute in the <a> element in the markup, but 'disabled' is not a valid attribute for <a> according to the HTML 4.01 specification or the XHTML 1.0 Transitional DTD (http://www.w3.org/TR/html4/loose.dtd and http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd respectively). The link is still clickable on both FF and IE.

      Notice the case of <ice:outputLink>. When using disabled="true", the 'href' attribute is not included in the <a> element, which makes the value of this component appear as plain text.

        Activity

        Arturo Zambrano created issue -
        Ken Fyten made changes -
        Field Original Value New Value
        Fix Version/s 1.7 [ 10080 ]
        Ken Fyten made changes -
        Assignee Adnan Durrani [ adnan.durrani ]
        Adnan Durrani made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #15217 Wed Nov 21 14:18:50 MST 2007 adnan.durrani Fix for ICE-2170 (disabled="true" not working as expected for <ice:commandLink>)
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/renderkit/dom_html_basic/CommandLinkRenderer.java
        Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/ext/renderkit/CommandLinkRenderer.java
        Hide
        Adnan Durrani added a comment -

        revision 15217

        Show
        Adnan Durrani added a comment - revision 15217
        Adnan Durrani made changes -
        Status In Progress [ 3 ] Resolved [ 5 ]
        Fix Version/s 1.7DR#3 [ 10112 ]
        Fix Version/s 1.7 [ 10080 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Priority Major [ 3 ] Minor [ 4 ]
        Hide
        Ajeet Sisodiya added a comment -

        We have already downloaded 1.7.0Beta1 version of iceface, but disable="true" functionality is not working into this.
        Please lets us the jar or the solution so that it should work.

        Show
        Ajeet Sisodiya added a comment - We have already downloaded 1.7.0Beta1 version of iceface, but disable="true" functionality is not working into this. Please lets us the jar or the solution so that it should work.
        Ken Fyten made changes -
        Fix Version/s 1.7 [ 10080 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Assignee Adnan Durrani [ adnan.durrani ]
        Hide
        Gabriel Gubis added a comment -

        See the specification of w3c on http://www.w3.org/TR/html401/interact/forms.html#adef-disabled... according to that statement, you can't put a disabled attribute to the <a> elements (only there mentioned are enabled), however the dommandLink is <a> generated element. It is not an icefaces bug.

        You can bypass this by changing the commandLink elements to commandButton (and setting the css style of the button so it will look like a link)

        Show
        Gabriel Gubis added a comment - See the specification of w3c on http://www.w3.org/TR/html401/interact/forms.html#adef-disabled ... according to that statement, you can't put a disabled attribute to the <a> elements (only there mentioned are enabled), however the dommandLink is <a> generated element. It is not an icefaces bug. You can bypass this by changing the commandLink elements to commandButton (and setting the css style of the button so it will look like a link)

          People

          • Assignee:
            Unassigned
            Reporter:
            Arturo Zambrano
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: