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

        Arran Mccullough created issue -
        Arran Mccullough made changes -
        Field Original Value New Value
        Attachment menuBar-issue.JPG [ 12139 ]
        Attachment Case8861Example.war [ 12140 ]
        Arran Mccullough made changes -
        Salesforce Case [5007000000Ap3Q8]
        Ken Fyten made changes -
        Fix Version/s 1.8.2-EE-GA_P01 [ 10220 ]
        Assignee Yip Ng [ yip.ng ]
        Ken Fyten made changes -
        Fix Version/s 1.8.3 [ 10211 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #20232 Thu Jan 14 14:12:07 MST 2010 yip.ng ICE-5251: Fixed nenuItem displaying in wrong position when located in a scrollable div.
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/bridge/component/menu.js
        yip.ng made changes -
        Attachment screenshot-1.png [ 12153 ]
        Hide
        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
        yip.ng added a comment -

        Fixed. See screenshot-1.

        Show
        yip.ng added a comment - Fixed. See screenshot-1.
        yip.ng made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        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.
        Arran Mccullough made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Hide
        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
        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.
        yip.ng made changes -
        Attachment screenshot-2.png [ 12188 ]
        Hide
        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.)
        Brad Kroeger made changes -
        Salesforce Case [5007000000Ap3Q8] [5007000000Ap3Q8, 5007000000ApQrS]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #20500 Fri Feb 05 13:28:15 MST 2010 yip.ng ICE-5251: Fixed nenuItem displaying in wrong position when located in a scrollable div.
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/bridge/component/menu.js
        Hide
        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
        yip.ng made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        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
        Joanne Bai made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Hide
        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
        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
        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: