ICEfaces
  1. ICEfaces
  2. ICE-5072

compat version of component-showcase panelPopup doesn't show modal popup when none-draggable.

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0-Alpha1
    • Fix Version/s: 2.0-Alpha2, 2.0.0
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      jsf2.0, ICEfaces2.0, compat

      Description

      No server log exception traces, but no popup gets displayed to close the modal Popup Message. The Draggable Popup Message does work properly though.

      I bet that the popup is just being positioned off the visible viewport for some reason.
      1. screenshot-1.png
        201 kB
      2. screenshot-2.png
        132 kB
      3. screenshot-3.png
        141 kB

        Issue Links

          Activity

          Judy Guglielmin created issue -
          Judy Guglielmin made changes -
          Field Original Value New Value
          Link This issue depends on ICE-4458 [ ICE-4458 ]
          Ken Fyten made changes -
          Summary compat version of component-showcase panelPopup doesn't show popup to close "A modal message." compat version of component-showcase panelPopup doesn't show modal popup when none-draggable.
          Salesforce Case []
          Fix Version/s 2.0 [ 10032 ]
          Assignee Priority P1
          Description No server log exception traces, but no popup gets displayed to close the modal Popup Message. The Draggable Popup Message does work properly though. No server log exception traces, but no popup gets displayed to close the modal Popup Message. The Draggable Popup Message does work properly though.

          I bet that the popup is just being positioned off the visible viewport for some reason.
          Security Private [ 10001 ]
          Assignee Yip Ng [ yip.ng ]
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #19579 Mon Nov 02 14:52:08 MST 2009 yip.ng ICE-5072: Fixed modal popup not appearing.
          Files Changed
          Commit graph MODIFY /icefaces/scratchpads/glimmer/compat/core/src/main/resources/META-INF/resources/ice-extras.js.txt
          Ken Fyten made changes -
          Fix Version/s 2.0-Alpha1 [ 10213 ]
          Fix Version/s 2.0-Beta [ 10032 ]
          Affects Version/s 2.0-Alpha1 [ 10213 ]
          Affects Version/s 2.0-Beta [ 10032 ]
          Ken Fyten made changes -
          Fix Version/s 2.0-Alpha2 [ 10214 ]
          Fix Version/s 2.0-Alpha1 [ 10213 ]
          yip.ng made changes -
          Attachment screenshot-1.png [ 12075 ]
          Hide
          yip.ng added a comment -

          Fixed. See screenshot-1.

          The popup was not off screen. It was hidden under the iframe. There are some event handler registrations being done in the popup JS. Somehow, using standard JS to do this will trip the JSF 2 JS, causing the popup JS to hang (or it seemed so when tracing using Firebug). Changing code to use Prototype API solved the problem.

          Show
          yip.ng added a comment - Fixed. See screenshot-1. The popup was not off screen. It was hidden under the iframe. There are some event handler registrations being done in the popup JS. Somehow, using standard JS to do this will trip the JSF 2 JS, causing the popup JS to hang (or it seemed so when tracing using Firebug). Changing code to use Prototype API solved the problem.
          yip.ng made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #19588 Tue Nov 03 13:09:00 MST 2009 yip.ng ICE-5072: Fixed modal popup not appearing.
          Files Changed
          Commit graph MODIFY /icefaces/scratchpads/glimmer/compat/core/src/main/resources/META-INF/resources/ice-extras.js
          yip.ng made changes -
          Attachment screenshot-2.png [ 12076 ]
          Hide
          yip.ng added a comment -

          Seems Mircea just replaced ice-extras.js with an old version, stomping over my changes. Changes re-done. See screenshot-2.

          See http://screencast.com/t/rCjTpdPaIH for the testing.

          Show
          yip.ng added a comment - Seems Mircea just replaced ice-extras.js with an old version, stomping over my changes. Changes re-done. See screenshot-2. See http://screencast.com/t/rCjTpdPaIH for the testing.
          yip.ng made changes -
          Attachment screenshot-3.png [ 12077 ]
          Hide
          yip.ng added a comment -

          In IE, there is a new bridge code error on the start page, preventing me from even getting to the popup dialog demo. page. See http://screencast.com/t/pvqzMoJdpc and screenshot-3.

          Show
          yip.ng added a comment - In IE, there is a new bridge code error on the start page, preventing me from even getting to the popup dialog demo. page. See http://screencast.com/t/pvqzMoJdpc and screenshot-3.
          Ken Fyten made changes -
          Fix Version/s 2.0.0 [ 10230 ]
          Ken Fyten made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Assignee Priority P1

            People

            • Assignee:
              yip.ng
              Reporter:
              Judy Guglielmin
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: