ICEfaces
  1. ICEfaces
  2. ICE-6645

BlockUIOnSubmit and toggleOnInput set to true prevents commandButtons and links from selecting row

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Won't Fix
    • Affects Version/s: 1.8.2-EE-GA_P02
    • Fix Version/s: EE-1.8.2.GA_P03
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      -
    • Workaround Exists:
      Yes
    • Workaround Description:
      Hide
      Workaround is to code the button or link component action listeners to perform the row selection state toggle for the row prior to whatever other processing they are doing. May be necessary to use toggleOnInput=false for this in some cases (where the app. itself would toggle row selection based on input element action events, etc.).
      Show
      Workaround is to code the button or link component action listeners to perform the row selection state toggle for the row prior to whatever other processing they are doing. May be necessary to use toggleOnInput=false for this in some cases (where the app. itself would toggle row selection based on input element action events, etc.).

      Description

      When setting BlockUIOnSubmit and toggleOnInput to true, you are unable to select a row in the dataTable when clicking on a commandButton or commandLink. InputText row selection works as expected. Test case attached.

        Activity

        Hide
        Ken Fyten added a comment -

        Possible that commandButton is firing the Action event, which causes blockUIOnSubmit to block, which prevents the subsequent submit for the rowSelection from occurring ?

        Show
        Ken Fyten added a comment - Possible that commandButton is firing the Action event, which causes blockUIOnSubmit to block, which prevents the subsequent submit for the rowSelection from occurring ?
        Hide
        Mircea Toma added a comment -

        The dataTable component wires up a 'onclick' event handler that eventually invokes iceSubmitPartial() function to inform the server about the newly selected row. When a commandButton or commandLink is used the 'onclick' event handlers of these components already invoke the iceSubmit() function. Because the blockUIOnSubmit feature blocks any subsequent requests from occurring the 'onclick' event handler wired to the table never gets a chance to send the new row position.

        The inputText components does work well because there's no 'onclick' event handler registered with the 'input' element. The 'click' event bubbles up to the table element which submits the new row position.

        Show
        Mircea Toma added a comment - The dataTable component wires up a 'onclick' event handler that eventually invokes iceSubmitPartial() function to inform the server about the newly selected row. When a commandButton or commandLink is used the 'onclick' event handlers of these components already invoke the iceSubmit() function. Because the blockUIOnSubmit feature blocks any subsequent requests from occurring the 'onclick' event handler wired to the table never gets a chance to send the new row position. The inputText components does work well because there's no 'onclick' event handler registered with the 'input' element. The 'click' event bubbles up to the table element which submits the new row position.
        Hide
        Ken Fyten added a comment -

        Resolving this issue within the components themselves would require a very large amount of effort and complexity/risk due to the nature of how these features must be implemented in the browser. Since a reasonable workaround is available, marking this as Won't Fix.

        Show
        Ken Fyten added a comment - Resolving this issue within the components themselves would require a very large amount of effort and complexity/risk due to the nature of how these features must be implemented in the browser. Since a reasonable workaround is available, marking this as Won't Fix.

          People

          • Assignee:
            Ken Fyten
            Reporter:
            Tyler Johnson
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: