ICEfaces
  1. ICEfaces
  2. ICE-3290

portlet: modal popup screen expanding problem

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 1.7.1
          • Fix Version/s: 1.8DR#2, 1.8
          • Component/s: ICE-Components
          • Labels:
            None
          • Environment:
            ANY (portlet and standalone component-showcase sample app).

            Description

            When displaying modal popup on the screen and scroll the mouse, the shadow area of the screen keeps expanding and also the popup window position keeps shifting.

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              Tyler Johnson added a comment -

              Issue reproducible with component showcase.

              Show
              Tyler Johnson added a comment - Issue reproducible with component showcase.
              Hide
              Permalink
              Ken Fyten added a comment -

              To reproduce the issue:

              1. Using stock component showcase, make the browser window shorter than the height required to display the contents of the app.
              2. On the popup dialog demo view create a modal popup.
              3. Once the modal dialog is displayed, scroll the page using the scrollbar, watch as the page length increases and the modal dialog re-positions with each scroll event.

              Seems like the logic for positing and sizing the modal dialog is also causing the page itself to grow slightly. If we could make it so the page wouldn't grow that would be a good fix.

              Show
              Ken Fyten added a comment - To reproduce the issue: 1. Using stock component showcase, make the browser window shorter than the height required to display the contents of the app. 2. On the popup dialog demo view create a modal popup. 3. Once the modal dialog is displayed, scroll the page using the scrollbar, watch as the page length increases and the modal dialog re-positions with each scroll event. Seems like the logic for positing and sizing the modal dialog is also causing the page itself to grow slightly. If we could make it so the page wouldn't grow that would be a good fix.
              Hide
              Permalink
              yip.ng added a comment -

              Fixed, including fix for Opera 9.5+. Tested on IE 6 & 7, FF 2 & 3, Opera pre-9.5 & 9.5+.

              There were actually several bugs (CSS, JavaScript, browser brand and browser version differences) involved, some of which might have been avoided in the first place had we made more use of Prototype functions rather than trying to "re-invent the wheel". (I have seen quite a few such cases in the past when fixing other JIRA cases.)

              Show
              yip.ng added a comment - Fixed, including fix for Opera 9.5+. Tested on IE 6 & 7, FF 2 & 3, Opera pre-9.5 & 9.5+. There were actually several bugs (CSS, JavaScript, browser brand and browser version differences) involved, some of which might have been avoided in the first place had we made more use of Prototype functions rather than trying to "re-invent the wheel". (I have seen quite a few such cases in the past when fixing other JIRA cases.)
              Hide
              Permalink
              Ken Fyten added a comment -

              The fix for this is causing regressions on the 1.7 branch, reverting for 1.7.2 release, will examine further for 1.8.

              Show
              Ken Fyten added a comment - The fix for this is causing regressions on the 1.7 branch, reverting for 1.7.2 release, will examine further for 1.8.

                People

                • Assignee:
                  Unassigned
                  Reporter:
                  Sam Xiao
                • Votes:
                  3 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: