ICEfaces
  1. ICEfaces
  2. ICE-11295

ice:menuBar/menuItem - sub menu items not shown in correct position in a scrollable browser window

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: EE-1.8.2.GA_P09
          • Fix Version/s: EE-1.8.2.GA_P10
          • Component/s: ICE-Components
          • Labels:
            None
          • Environment:
            All browsers

            Description

            An ice:menuBar is setup in a browser window that scrolls. When displaying the 3rd level menuItem, the positioning is off when the browser has been scrolled down. If it isn't scrolled, the positioning is correct.

            This seems to be an issue in Firefox, Chrome and IE11. I also tested with scrollableDivMode = true on the menuBar but it had no effect.

              Activity

              Ascending order - Click to sort in descending order
              Arran Mccullough created issue -
              Hide
              Permalink
              Arran Mccullough added a comment -

              Attached test case to reproduce the issue.

              Steps:

              • Load welcomeICEfaces.iface
              • Scroll down the browser slightly.
              • Hover over the "My Home" menuBar.
              • Display the child menu of one of the displayed menu items. It should be displayed disjointed from the menu.
              Show
              Arran Mccullough added a comment - Attached test case to reproduce the issue. Steps: Load welcomeICEfaces.iface Scroll down the browser slightly. Hover over the "My Home" menuBar. Display the child menu of one of the displayed menu items. It should be displayed disjointed from the menu.
              Arran Mccullough made changes -
              Field Original Value New Value
              Attachment Case14096Example.war [ 22512 ]
              Attachment Case14096Example.zip [ 22513 ]
              Hide
              Permalink
              Ken Fyten added a comment -

              Note that the EE 3.3 version of the component does not have this issue, possible it was previously resolved but not backported to EE 1.8.2?

              Show
              Ken Fyten added a comment - Note that the EE 3.3 version of the component does not have this issue, possible it was previously resolved but not backported to EE 1.8.2?
              Ken Fyten made changes -
              Assignee Arturo Zambrano [ artzambrano ]
              Fix Version/s EE-1.8.2.GA_P10 [ 13089 ]
              Assignee Priority P1 [ 10010 ]
              Hide
              Permalink
              Arturo Zambrano added a comment - - edited

              r51516: removed unnecessary offsets when calculating submenu height on the page, since the height is already correctly calculated from the parent submenu

              It was a different issue.

              Testing notes: please test with all supported browsers.

              Show
              Arturo Zambrano added a comment - - edited r51516: removed unnecessary offsets when calculating submenu height on the page, since the height is already correctly calculated from the parent submenu It was a different issue. Testing notes: please test with all supported browsers.
              Arturo Zambrano made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #51516 Thu May 11 14:56:12 MDT 2017 art.zambrano ICE-11295 removed unnecessary offsets when calculating submenu height on the page, since the height is already correctly calculated from the parent submenu
              Files Changed
              Commit graph MODIFY /icefaces/trunk/icefaces/bridge/component/menu.js
              Hide
              Permalink
              Liana Munroe added a comment -

              Verified ICEfaces 1.8.2 trunk r51524 using attached test case. Tomcat 8, MS Edge, IE 11, 10, 9, 8, 7, FF 47, Chrome 57.

              Show
              Liana Munroe added a comment - Verified ICEfaces 1.8.2 trunk r51524 using attached test case. Tomcat 8, MS Edge, IE 11, 10, 9, 8, 7, FF 47, Chrome 57.
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

                • Assignee:
                  Arturo Zambrano
                  Reporter:
                  Arran Mccullough
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  4 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: