ICEfaces
  1. ICEfaces
  2. ICE-5251

MenuItem displays in wrong position when located in a scrollable div

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 1.8.2-EE-GA
          • Fix Version/s: 1.8.2-EE-GA_P01, 1.8.3
          • Component/s: ICE-Components
          • Labels:
            None
          • Environment:
            All

            Description

            When a menuBar is located in a scrollable div the menuItem's are shown in the wrong position. They are only displayed in the wrong position when menuBar is in the lower part of the scrollable div.

              Activity

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

              The issue is partly caused by bugs in the fix for ICE-4106 and partly caused by bugs in Prototype's viewportOffset() function.

              Show
              yip.ng added a comment - The issue is partly caused by bugs in the fix for ICE-4106 and partly caused by bugs in Prototype's viewportOffset() function.
              Hide
              Permalink
              yip.ng added a comment -

              Fixed. See screenshot-1.

              Show
              yip.ng added a comment - Fixed. See screenshot-1.
              Hide
              Permalink
              Arran Mccullough added a comment -

              The customer has reported that the fix doesn't work in IE. I've confirmed this by testing the latest trunk and the patched build provided to the customer with my test case.

              Show
              Arran Mccullough added a comment - The customer has reported that the fix doesn't work in IE. I've confirmed this by testing the latest trunk and the patched build provided to the customer with my test case.
              Hide
              Permalink
              yip.ng added a comment - - edited

              Can't reproduce. It's working in IE. See video at http://screencast.com/t/ZDlhOGJlMDkt.

              Show
              yip.ng added a comment - - edited Can't reproduce. It's working in IE. See video at http://screencast.com/t/ZDlhOGJlMDkt .
              Hide
              Permalink
              yip.ng added a comment -

              Same with 1.8.2 EE. Purposely set version numbers to make sure test app. was run against that branch. See screenshot-2.

              Show
              yip.ng added a comment - Same with 1.8.2 EE. Purposely set version numbers to make sure test app. was run against that branch. See screenshot-2.
              Hide
              Permalink
              yip.ng added a comment -

              Doesn't work in IE6 or the IE7 debug mode in IE8. And the problem seems to be different in each case. (We are already using Prototype functions, which are supposed to be cross-browser.)

              Show
              yip.ng added a comment - Doesn't work in IE6 or the IE7 debug mode in IE8. And the problem seems to be different in each case. (We are already using Prototype functions, which are supposed to be cross-browser.)
              Hide
              Permalink
              yip.ng added a comment -

              Revision: 20500

              Modified : /icefaces/trunk/icefaces/bridge/component/menu.js

              Revision: 20499
              Author: yip.ng
              Date: 1:27:31 PM, February 5, 2010
              Message:
              ICE-5251: Fixed menuItem displaying in wrong position when located in a scrollable div.

              Modified : /icefaces-ee/branches/icefaces-ee-1.8.2/icefaces/bridge/component/menu.js

              Show
              yip.ng added a comment - Revision: 20500 Modified : /icefaces/trunk/icefaces/bridge/component/menu.js Revision: 20499 Author: yip.ng Date: 1:27:31 PM, February 5, 2010 Message: ICE-5251 : Fixed menuItem displaying in wrong position when located in a scrollable div. Modified : /icefaces-ee/branches/icefaces-ee-1.8.2/icefaces/bridge/component/menu.js
              Hide
              Permalink
              Joanne Bai added a comment -

              Verified with success during release testing cycle. Test app is available at repo\qa\trunk\Regression\ICE-5251

              Show
              Joanne Bai added a comment - Verified with success during release testing cycle. Test app is available at repo\qa\trunk\Regression\ ICE-5251
              Hide
              Permalink
              Isuru Perera added a comment -

              There is an issue. Please check my forum post http://www.icefaces.org/JForum/posts/list/0/16781.page#62516

              Show
              Isuru Perera added a comment - There is an issue. Please check my forum post http://www.icefaces.org/JForum/posts/list/0/16781.page#62516
              Hide
              Permalink
              Donald Rumac added a comment -

              This issue is only visible in IE7, not in IE8 for example.

              Show
              Donald Rumac added a comment - This issue is only visible in IE7, not in IE8 for example.
              Hide
              Permalink
              Stefan Friedrich added a comment -

              Any update on this issue?

              We're experiencing the same with ICEFaces EE 1.8.2_P02

              Show
              Stefan Friedrich added a comment - Any update on this issue? We're experiencing the same with ICEFaces EE 1.8.2_P02

                People

                • Assignee:
                  yip.ng
                  Reporter:
                  Arran Mccullough
                • Votes:
                  1 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: