ICEfaces
  1. ICEfaces
  2. ICE-6168

Regression: Modal Popup has problem on IE7 & IE6

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0-Beta2
    • Fix Version/s: 2.0.0
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      IE7 & IE6
      Icefaces2 revision# 22877
      server: Tomcat6

      Description

      Compat component-showcase Modal popup does not work properly on IE7 browser.
      Click on show modal popup button.
      The styling changes on the page but no Modal popup is displayed.

      Note: It works fine on IE8
                On IE6 although modal popup is displayed the styling has some issue (some code is displayed in the background)

        Activity

        Mandeep Hayher created issue -
        Mandeep Hayher made changes -
        Field Original Value New Value
        Assignee Ken Fyten [ ken.fyten ]
        Ken Fyten made changes -
        Salesforce Case []
        Fix Version/s 2.0-Beta2 [ 10242 ]
        Assignee Priority P1
        Assignee Ken Fyten [ ken.fyten ] Yip Ng [ yip.ng ]
        Hide
        yip.ng added a comment -

        I get an error message:

        Line: 17165
        Error: 'style' is null or not an object

        Need to debug the JavaScript.

        Show
        yip.ng added a comment - I get an error message: Line: 17165 Error: 'style' is null or not an object Need to debug the JavaScript.
        Hide
        yip.ng added a comment -

        Prototype is accessing the style object on the document element, which returns undefined in IE7. But it works fine in 1.8 component showcase. Why?

        Show
        yip.ng added a comment - Prototype is accessing the style object on the document element, which returns undefined in IE7. But it works fine in 1.8 component showcase. Why?
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #22967 Thu Oct 28 08:49:21 MDT 2010 yip.ng ICE-6168: Regression: Modal Popup has problem on IE7 & IE6.
        Files Changed
        Commit graph MODIFY /icefaces2/trunk/icefaces/compat/core/src/main/javascript/prototype/prototype.js
        Hide
        Ken Fyten added a comment -

        I cannot reproduce this issue on IE7 or IE8 with cleared caches.

        Show
        Ken Fyten added a comment - I cannot reproduce this issue on IE7 or IE8 with cleared caches.
        Hide
        yip.ng added a comment -

        I can consistently reproduce it on my machine. Anyways, I have checked in a quick fix. But I still can't explain the difference in behavior between showcase 1.8 and compat.

        Show
        yip.ng added a comment - I can consistently reproduce it on my machine. Anyways, I have checked in a quick fix. But I still can't explain the difference in behavior between showcase 1.8 and compat.
        yip.ng made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Mandeep Hayher added a comment - - edited

        Tested successfully on icefaces2 trunk revision# 23003, tomcat6 & IE7 browser.
        But the test is failing on icefaces2 trunk revision# 23003, tomcat6 & IE6 browser.
        See attached screenshot-1.

        Show
        Mandeep Hayher added a comment - - edited Tested successfully on icefaces2 trunk revision# 23003, tomcat6 & IE7 browser. But the test is failing on icefaces2 trunk revision# 23003, tomcat6 & IE6 browser. See attached screenshot-1.
        Mandeep Hayher made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Mandeep Hayher made changes -
        Attachment screenshot-1.jpg [ 12648 ]
        Ken Fyten made changes -
        Salesforce Case []
        Fix Version/s 2.0.0 [ 10230 ]
        Fix Version/s 2.0-Beta2 [ 10242 ]
        Assignee Priority P1 P2
        Ken Fyten made changes -
        Salesforce Case []
        Fix Version/s 2.0.0-EE-Beta1 [ 10250 ]
        Fix Version/s 2.0.0 [ 10230 ]
        Assignee Priority P2
        Hide
        Ken Fyten added a comment -

        Please retest with latest icefaces2/trunk.

        Show
        Ken Fyten added a comment - Please retest with latest icefaces2/trunk.
        Ken Fyten made changes -
        Assignee Yip Ng [ yip.ng ] Mandeep Hayher [ mandeep.hayher ]
        Hide
        Mandeep Hayher added a comment -

        The test is still failing on icefaces2 trunk revision# 23277, tomcat6 & IE6 browser.

        Show
        Mandeep Hayher added a comment - The test is still failing on icefaces2 trunk revision# 23277, tomcat6 & IE6 browser.
        Mandeep Hayher made changes -
        Assignee Mandeep Hayher [ mandeep.hayher ] Ken Fyten [ ken.fyten ]
        Hide
        Srdjan Djordjic added a comment -

        The test is failing on GlassFish Mojarra 2.0.3 hosted on OpenSuse Linux. The IE version is 8. Panel Popup is created on button click, and it is displayed properly, but pre tag contains blank page from the screenshot is rendered. In situation when GF is hosted on Windows 7, background is displayed as it should be.

        Show
        Srdjan Djordjic added a comment - The test is failing on GlassFish Mojarra 2.0.3 hosted on OpenSuse Linux. The IE version is 8. Panel Popup is created on button click, and it is displayed properly, but pre tag contains blank page from the screenshot is rendered. In situation when GF is hosted on Windows 7, background is displayed as it should be.
        Hide
        Mandeep Hayher added a comment -

        Tested Successfully on ICEfaces2 revision# 23643.
        on Glassfishv3 + WindowsXP + IE8
        on Glassfishv3 + Linux + IE8

        Show
        Mandeep Hayher added a comment - Tested Successfully on ICEfaces2 revision# 23643. on Glassfishv3 + WindowsXP + IE8 on Glassfishv3 + Linux + IE8
        Mandeep Hayher made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Fix Version/s 2.0.0 [ 10230 ]
        Fix Version/s 2.0.0-EE-Beta1 [ 10250 ]
        Resolution Fixed [ 1 ]
        Mandeep Hayher made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Ken Fyten
            Reporter:
            Mandeep Hayher
          • Votes:
            1 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: