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

        Arran Mccullough created issue -
        Arran Mccullough made changes -
        Field Original Value New Value
        Salesforce Case [50070000008pM3u]
        Hide
        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
        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
        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
        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
        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: