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

              Ascending order - Click to sort in descending order
              Arran Mccullough created issue -
              Arran Mccullough made changes -
              Field Original Value New Value
              Salesforce Case [5007000000GuXp4]
              Arran Mccullough made changes -
              Attachment testcase-hideAll.JPG [ 13230 ]
              Attachment testcase-showBoth.JPG [ 13231 ]
              Attachment Case10109Example.war [ 13232 ]
              Hide
              Permalink
              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.
              Ken Fyten made changes -
              Fix Version/s EE-1.8.2.GA_P03 [ 10251 ]
              Fix Version/s 2.1 [ 10241 ]
              Fix Version/s EE-2.1.0 [ 10262 ]
              Assignee Priority P1
              Assignee Mircea Toma [ mircea.toma ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #24623 Thu May 19 07:51:27 MDT 2011 mircea.toma ICE-6868 Remove the modal popup regardless of its place in the stack of rendered popups.
              Files Changed
              Commit graph MODIFY /icefaces/trunk/icefaces/bridge/lib/extras/style.js
              Hide
              Permalink
              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.
              Mircea Toma made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Hide
              Permalink
              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.
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #24635 Fri May 20 12:34:48 MDT 2011 mircea.toma ICE-6868 Changed usage of Hash variable to Hashtable.
              Files Changed
              Commit graph MODIFY /icefaces/trunk/icefaces/bridge/lib/prototype/dom.js
              Hide
              Permalink
              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.
              Mandeep Hayher made changes -
              Status Resolved [ 5 ] Closed [ 6 ]
              Assignee Mircea Toma [ mircea.toma ] Ken Fyten [ ken.fyten ]

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: