ICEfaces
  1. ICEfaces
  2. ICE-9049

Fail to hide all popups (diffConfig=att specific)

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.2
    • Fix Version/s: 3.3
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      All supported browsers

      Description

      There's a regression when running ICE-6868 test using diffConfig=att, both popups are not closed when clicking on "hide all" button.

        Activity

        Hide
        Mircea Toma added a comment -

        This regression is caused by the popup1 which disables the event handlers of all elements, including popup2's, with the exception of the elements included in popup1's container. When attribute update is not enabled the event handlers are restored when the entire popup2 markup is updated, thus making the "hide all" button work again. The fix we need to apply in this case is at component level, not in JSF or our bridge

        Show
        Mircea Toma added a comment - This regression is caused by the popup1 which disables the event handlers of all elements, including popup2's, with the exception of the elements included in popup1's container. When attribute update is not enabled the event handlers are restored when the entire popup2 markup is updated, thus making the "hide all" button work again. The fix we need to apply in this case is at component level, not in JSF or our bridge
        Hide
        Mircea Toma added a comment -

        Modified modal popup JS code to restore the callbacks disabled by a previously opened popup so that only that the input elements of the last opened popup are active. The reverse was done when closing the last openend popup, the callbacks of the previously opened popup are re-disabled.

        Show
        Mircea Toma added a comment - Modified modal popup JS code to restore the callbacks disabled by a previously opened popup so that only that the input elements of the last opened popup are active. The reverse was done when closing the last openend popup, the callbacks of the previously opened popup are re-disabled.

          People

          • Assignee:
            Mircea Toma
            Reporter:
            Mircea Toma
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: