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

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 2.0-Alpha1
  • Fix Version/s: 2.0-Alpha2, 2.0.0
  • Component/s: 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
    02/Nov/09 5:00 PM
  2. screenshot-2.png
    132 kB
    03/Nov/09 3:25 PM
  3. screenshot-3.png
    141 kB
    03/Nov/09 3:52 PM

Issue Links

depends on

New Feature - A new feature of the product, which has yet to be developed. ICE-4458 create component-showcase sample for compat version of ICEfaces-2

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
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.
Hide
Permalink
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.
Hide
Permalink
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.

People

  • Assignee:
    yip.ng
    Reporter:
    Judy Guglielmin
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: