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
              Sam Xiao created issue -
              Ken Fyten made changes -
              Field Original Value New Value
              Salesforce Case []
              Fix Version/s 1.8 [ 10161 ]
              Assignee Priority P1
              Affects Version/s 1.8 [ 10161 ]
              Affects Version/s 1.8RC2 [ 10163 ]
              Security Private [ 10001 ]
              Assignee Yip Ng [ yip.ng ]
              Ken Fyten made changes -
              Environment IE 7 IE 7, ICEfaces 1.8.0-RC2 (RC1 and prior do not exhibit this problem).
              Salesforce Case []
              Affects Version/s 1.8RC2 [ 10163 ]
              Affects Version/s 1.8 [ 10161 ]
              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.
              Ken Fyten made changes -
              Workaround Description The problem disappears when com.icesoft.faces.blockUIOnSubmit is false in web.xml.
              Workaround Exists [Yes]
              Salesforce Case []
              Assignee Priority P1 P2
              Assignee Yip Ng [ yip.ng ] Mircea Toma [ mircea.toma ]
              Ken Fyten made changes -
              Summary REGRESSION: clicking selectoneradio label caused IE 7 page reload REGRESSION: clicking ice:selectOneRadio label caused IE 7 page reload
              Salesforce Case []
              Component/s Bridge [ 10011 ]
              Component/s Components [ 10012 ]
              Security Private [ 10001 ]
              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.
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #18697 Tue Mar 31 07:54:48 MDT 2009 mircea.toma ICE-4286 Use predefined functions that calculate correctly in most of browsers the window dimensions.
              Files Changed
              Commit graph MODIFY /icefaces/trunk/icefaces/bridge/src/status.js
              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.
              Mircea Toma made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              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.
              Ken Fyten made changes -
              Resolution Fixed [ 1 ]
              Status Resolved [ 5 ] Reopened [ 4 ]
              Assignee Priority P2 P1
              Ken Fyten made changes -
              Summary REGRESSION: clicking ice:selectOneRadio label caused IE 7 page reload REGRESSION: UI Blocker causes scrollbars to "flash" while it is displayed
              Salesforce Case []
              Ken Fyten made changes -
              Summary REGRESSION: UI Blocker causes scrollbars to "flash" while it is displayed UI Blocker causes scrollbars to "flash" while it is displayed
              Salesforce Case []
              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?
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #18703 Tue Mar 31 12:25:59 MDT 2009 yip.ng ICE-4286
              Reverted changes.
              Files Changed
              Commit graph MODIFY /icefaces/trunk/icefaces/bridge/src/status.js
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #18704 Tue Mar 31 12:46:35 MDT 2009 yip.ng ICE-4286
              Reduced size of iframe to prevent scrollbars flashing.
              Files Changed
              Commit graph MODIFY /icefaces/trunk/icefaces/bridge/src/status.js
              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.
              Mircea Toma made changes -
              Status Reopened [ 4 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Ken Fyten made changes -
              Assignee Priority P1
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]
              Assignee Mircea Toma [ mircea.toma ]

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: