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
              Arran Mccullough created issue -
              Arran Mccullough made changes -
              Field Original Value New Value
              Salesforce Case [50070000008pM3u]
              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.
              Arran Mccullough made changes -
              Attachment Case8375Example.war [ 11793 ]
              Ken Fyten made changes -
              Fix Version/s 1.8.2 [ 10190 ]
              Assignee Priority P1
              Affects Version/s 1.8.2 [ 10190 ]
              Affects Version/s 1.8.1 [ 10170 ]
              Assignee Adnan Durrani [ adnan.durrani ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #18973 Wed Jun 17 15:57:39 MDT 2009 adnan.durrani Fix for ICE-4595 (Modal background for popup not appearing with fix for ICE-4475 )
              Ice.modal.stop being sent only in renderResponse Phase.
              Files Changed
              Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/panelpopup/PanelPopup.java
              Adnan Durrani made changes -
              Status Open [ 1 ] In Progress [ 3 ]
              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
              Adnan Durrani made changes -
              Status In Progress [ 3 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              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
              Ken Fyten made changes -
              Fix Version/s 1.8.2-RC1 [ 10210 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #19949 Thu Dec 10 15:45:41 MST 2009 judy.guglielmin ICE-4481, update PanelPopup for ICE-5062, ICE-4778, ICE-4781, ICE-4700, ICE-4710, ICE-4578, ICE-4595, ICE-4475, ICE-3356, ICE-4495,ICE-4450
              Files Changed
              Commit graph MODIFY /icefaces/scratchpads/glimmer/compat/components/src/main/java/com/icesoft/faces/component/panelpopup/PanelPopup.java
              Commit graph MODIFY /icefaces/scratchpads/glimmer/compat/component-metadata/src/main/resources/conf/ice_cust_properties/cust-panelpopup-props.xml
              Commit graph MODIFY /icefaces/scratchpads/glimmer/compat/core/src/main/resources/META-INF/resources/ice-extras.js
              Commit graph MODIFY /icefaces/scratchpads/glimmer/compat/components/src/main/java/com/icesoft/faces/component/panelpopup/PanelPopupRenderer.java
              Commit graph MODIFY /icefaces/scratchpads/glimmer/compat/components/src/main/java/com/icesoft/faces/component/outputconnectionstatus/OutputConnectionStatusRenderer.java
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]
              Assignee Priority P1

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: