ICEfaces
  1. ICEfaces
  2. ICE-5371

menuBar displays outside of the browser view on first opening

    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
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      All

      Description

      When a long menuBar (has many menuItems) is placed near the top of the page and the browser is resized to a smaller size the menuBar is displayed above first menuItem. This makes it difficult to choose an item. If the menuBar is opened again then it is shown in the proper position. When the menuBar is first shown it seems to be seeing that the bottom of the browser view will interfere with the positioning so it displays it upwards. On the second time it seems to factor in the top and bottom of the browser view and displays it properly.

        Activity

        Hide
        Arran Mccullough added a comment -

        Attachments:

        • menuBar-issue-first.jpg: Shows the first rendering of the menuBar
        • menuBar-issue-second.jpg: Shows the second rendering of the menuBar.
        • Case8938Example.war: Basic test case.

        Note: This occurs in both IE and Firefox browsers.

        Show
        Arran Mccullough added a comment - Attachments: menuBar-issue-first.jpg: Shows the first rendering of the menuBar menuBar-issue-second.jpg: Shows the second rendering of the menuBar. Case8938Example.war: Basic test case. Note: This occurs in both IE and Firefox browsers.
        Hide
        Deryk Sinotte added a comment -

        Re-assigning to Ken for component team. Look at addressing for next release.

        Show
        Deryk Sinotte added a comment - Re-assigning to Ken for component team. Look at addressing for next release.
        Hide
        Ken Fyten added a comment -

        Not sure if this is a matter of recalcing the layout when the browser window/viewport resizes, or what?

        Show
        Ken Fyten added a comment - Not sure if this is a matter of recalcing the layout when the browser window/viewport resizes, or what?
        Hide
        yip.ng added a comment - - edited

        Can't reproduce. See video at http://screencast.com/t/ZTUyYjZm (App. started from within Intellij.)

        Show
        yip.ng added a comment - - edited Can't reproduce. See video at http://screencast.com/t/ZTUyYjZm (App. started from within Intellij.)
        Hide
        yip.ng added a comment - - edited

        Can't reproduce in FF. See above video. In IE, a different problem shows up. See video at http://screencast.com/t/YzdkMzdjM

        Show
        yip.ng added a comment - - edited Can't reproduce in FF. See above video. In IE, a different problem shows up. See video at http://screencast.com/t/YzdkMzdjM
        Hide
        yip.ng added a comment - - edited

        This is yet another bug introduced by ICE-4106. (I have fixed at least one other time-consuming bug before.) ICE-4106 included a new function (complicated) to replace the Prototype clonePosition function for IE. By reverting back to using the Prototype function at a certain place (where fix for ICE-3658 was added), the current problem is solved. See video at http://screencast.com/t/NDJkMWYwNDkt

        Show
        yip.ng added a comment - - edited This is yet another bug introduced by ICE-4106 . (I have fixed at least one other time-consuming bug before.) ICE-4106 included a new function (complicated) to replace the Prototype clonePosition function for IE. By reverting back to using the Prototype function at a certain place (where fix for ICE-3658 was added), the current problem is solved. See video at http://screencast.com/t/NDJkMWYwNDkt
        Hide
        yip.ng added a comment -

        Revision: 20887


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

        Revision: 20888


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

        Show
        yip.ng added a comment - Revision: 20887 Modified : /icefaces/trunk/icefaces/bridge/component/menu.js Revision: 20888 Modified : /icefaces-ee/branches/icefaces-ee-1.8.2/icefaces/bridge/component/menu.js
        Hide
        Joanne Bai added a comment -

        Verified successfully on ICEfaces-1.8.2-ee revision 20900 using tomcat 6 + FF3.5 and IE8

        Test app has been committed to repo\qa\trunk\Regression\ICE-5371

        Show
        Joanne Bai added a comment - Verified successfully on ICEfaces-1.8.2-ee revision 20900 using tomcat 6 + FF3.5 and IE8 Test app has been committed to repo\qa\trunk\Regression\ ICE-5371

          People

          • Assignee:
            Joanne Bai
            Reporter:
            Arran Mccullough
          • Votes:
            1 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: