ICEfaces
  1. ICEfaces
  2. ICE-4595

Modal background for popup not appearing with fix for ICE-4475

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 1.8.2
          • Fix Version/s: 1.8.2-RC1, 1.8.2
          • Component/s: ICE-Components
          • Labels:
            None
          • Environment:
            All
          • Workaround Exists:
            Yes
          • Workaround Description:
            Use the visible attribute to show/hide the popup

            Description

            The fix that was implemented for ICE-4475 is causing a modal popups modal background to not display. This issue only happens when the popup is on the same form as the button(or other mechanism) that shows the popup and is using its rendered attribute to show/hide the popup. If the popup is in a different form then the modal background shows fine.

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              Arran Mccullough added a comment -

              I've tested with the trunk (Rev. 18953) and also adding the fix for ICE-4475 to ICEfaces 1.8.1

              Show
              Arran Mccullough added a comment - I've tested with the trunk (Rev. 18953) and also adding the fix for ICE-4475 to ICEfaces 1.8.1
              Hide
              Permalink
              Arran Mccullough added a comment -

              Test case that has two commandButtons. Button #1 opens a popup in the same form. Button #2 opens a popup in a different form.

              Show
              Arran Mccullough added a comment - Test case that has two commandButtons. Button #1 opens a popup in the same form. Button #2 opens a popup in a different form.
              Hide
              Permalink
              Adnan Durrani added a comment -

              Fixed.

              Modified: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\panelpopup\PanelPopup.java
              Sending content: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\panelpopup\PanelPopup.java
              Completed: At revision: 18973

              Show
              Adnan Durrani added a comment - Fixed. Modified: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\panelpopup\PanelPopup.java Sending content: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\panelpopup\PanelPopup.java Completed: At revision: 18973
              Hide
              Permalink
              Krashan Brahmanjara added a comment -

              It's still doesn't work.

              Cascaded modal popups are rendered without background iframe (or background frame is disabled by js?)

              My case.
              One jspx with few modal popups included. Each are rendererd on demand.
              First I render and enable one then on button command I render second on top. This second popup is without iframe and bottom popup is accesiible for editing.

              Show
              Krashan Brahmanjara added a comment - It's still doesn't work. Cascaded modal popups are rendered without background iframe (or background frame is disabled by js?) My case. One jspx with few modal popups included. Each are rendererd on demand. First I render and enable one then on button command I render second on top. This second popup is without iframe and bottom popup is accesiible for editing.
              Hide
              Permalink
              Krashan Brahmanjara added a comment -

              Case8375Example.war working correctly now but problem described by me is silmilar and was moved to ICE-4734

              Show
              Krashan Brahmanjara added a comment - Case8375Example.war working correctly now but problem described by me is silmilar and was moved to ICE-4734

                People

                • Assignee:
                  Adnan Durrani
                  Reporter:
                  Arran Mccullough
                • Votes:
                  1 Vote for this issue
                  Watchers:
                  2 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: