ICEfaces
  1. ICEfaces
  2. ICE-6868

modal background isn't removed when closing multiple modal popups

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.8.2-EE-GA_P02
    • Fix Version/s: EE-1.8.2.GA_P03, 3.0, EE-3.0.0.GA
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      All, panelPopup using visible attribute
    • Assignee Priority:
      P1
    • Workaround Exists:
      Yes
    • Workaround Description:
      Use the rendered attribute instead.

      Description

      When using the visible attribute on the panelPopup component to show/hide the popups, opening multiple modal popups and then closing all of these popups causes one of the modal popups modal backgrounds to not be removed.

        Activity

        Hide
        Krzysztof Dziankowski added a comment -

        1.8.2 version is also affected. As a work-around we can use rendered instead of visible.

        Show
        Krzysztof Dziankowski added a comment - 1.8.2 version is also affected. As a work-around we can use rendered instead of visible.
        Hide
        Mircea Toma added a comment -

        The issue occurs when the 2 modal popups are removed from the list of rendered popups in a different order the reverse order of their creation. Initially the modal popup handling code was written to work with a stack of popups where the first added popup is the last one out. along the time the code was changed to work with just a free list of popups to allow more flexibility.

        Show
        Mircea Toma added a comment - The issue occurs when the 2 modal popups are removed from the list of rendered popups in a different order the reverse order of their creation. Initially the modal popup handling code was written to work with a stack of popups where the first added popup is the last one out. along the time the code was changed to work with just a free list of popups to allow more flexibility.
        Hide
        Mircea Toma added a comment -

        Changed modal popup handling to remove the modal popup regardless of its place in the stack of rendered popups.

        Show
        Mircea Toma added a comment - Changed modal popup handling to remove the modal popup regardless of its place in the stack of rendered popups.
        Hide
        Mandeep Hayher added a comment -

        Tested successfully on Icefaces1.8 trunk revision# 24679, Tomcat6 using FF3.6, IE8 & Chrome10 browsers.

        Show
        Mandeep Hayher added a comment - Tested successfully on Icefaces1.8 trunk revision# 24679, Tomcat6 using FF3.6, IE8 & Chrome10 browsers.

          People

          • Assignee:
            Ken Fyten
            Reporter:
            Arran Mccullough
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: