ICEfaces
  1. ICEfaces
  2. ICE-4286

UI Blocker causes scrollbars to "flash" while it is displayed

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 1.8RC2
          • Fix Version/s: 1.8
          • Component/s: Bridge
          • Labels:
            None
          • Environment:
            IE 7, ICEfaces 1.8.0-RC2 (RC1 and prior do not exhibit this problem).
          • Workaround Exists:
            Yes
          • Workaround Description:
            The problem disappears when com.icesoft.faces.blockUIOnSubmit is false in web.xml.

            Description

            When verify ICE-2175 in IE 7 browser, clicking the radio label caused full page update which is not present in firefox.

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              yip.ng added a comment -

              The "regression" has nothing to do with ICE-2175. It started at revision 18592, which was changes by ICE-4203.

              Show
              yip.ng added a comment - The "regression" has nothing to do with ICE-2175 . It started at revision 18592, which was changes by ICE-4203 .
              Hide
              Permalink
              yip.ng added a comment -

              The problem disappears when com.icesoft.faces.blockUIOnSubmit is false in web.xml.

              Show
              yip.ng added a comment - The problem disappears when com.icesoft.faces.blockUIOnSubmit is false in web.xml.
              Hide
              Permalink
              Mircea Toma added a comment -

              IE7 only looks like is doing a full page load. Looking in the bridge log you can see exactly the same updates sent for IE7 or FF.
              It looks like that the overlay iframe when rendered is causing the scroll bars to briefly show up. This is purely a styling issue.

              Show
              Mircea Toma added a comment - IE7 only looks like is doing a full page load. Looking in the bridge log you can see exactly the same updates sent for IE7 or FF. It looks like that the overlay iframe when rendered is causing the scroll bars to briefly show up. This is purely a styling issue.
              Hide
              Permalink
              Mircea Toma added a comment -

              Use predefined functions to calculate correctly (in most of browsers) the window dimensions.

              Show
              Mircea Toma added a comment - Use predefined functions to calculate correctly (in most of browsers) the window dimensions.
              Hide
              Permalink
              Ken Fyten added a comment -

              Last change is causing regressions in FF3. Now scrollbars flash in FF3 all the time (if they are not already displayed).

              Also, it takes noticeably longer to display/hide the hourglass and UI blocker using this approach.

              Suggest we back off the JS resizing logic and revert to previous implementation as it was generally more acceptable.

              Show
              Ken Fyten added a comment - Last change is causing regressions in FF3. Now scrollbars flash in FF3 all the time (if they are not already displayed). Also, it takes noticeably longer to display/hide the hourglass and UI blocker using this approach. Suggest we back off the JS resizing logic and revert to previous implementation as it was generally more acceptable.
              Hide
              Permalink
              Ken Fyten added a comment -

              Can we make the initial size of the UI Blocker a little smaller than the viewport so that we don't risk having the scrollbars flash?

              Show
              Ken Fyten added a comment - Can we make the initial size of the UI Blocker a little smaller than the viewport so that we don't risk having the scrollbars flash?
              Hide
              Permalink
              yip.ng added a comment -

              Changes to status.js reverted.

              Show
              yip.ng added a comment - Changes to status.js reverted.
              Hide
              Permalink
              yip.ng added a comment -

              Reduced size of iframe to prevent scrollbars flashing.

              Show
              yip.ng added a comment - Reduced size of iframe to prevent scrollbars flashing.
              Hide
              Permalink
              Mircea Toma added a comment -

              Fixed by Yip.

              Show
              Mircea Toma added a comment - Fixed by Yip.

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: