ICEfaces
  1. ICEfaces
  2. ICE-5945

Regression Icefaces2 - MenuBar & Menupop submenus are not displayed

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 2.0-Beta2
          • Fix Version/s: 2.0-Beta2
          • Component/s: ICE-Components
          • Labels:
            None
          • Environment:
            Server: Tomcat6
            Browser: FF3.6, IE7 & Opera10.10

            Description

            Icefaces2 revision#22029

            Submenus under MenuBar and MenuPopup are not being displayed. The problem can be reproduced on compat component-showcase. Other test cases failing due to this problem are: ICE-3856, ICE-3320, ICE-1232

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              Deryk Sinotte added a comment -

              The issue in both cases appears to be related to this JavaScript error:

              $ is not defined

              This is due to the compat component re-sync (ICE-5665) being done after the work to hide Prototype functions (ICE-5844). The re-sync reverted the Prototype hiding work which causes the JavaScript functions to not be found. I fixed up the MenuItemRenderer class but we there was overlap between the two different changes so we'll need to check to see if anything else got reverted.

              Show
              Deryk Sinotte added a comment - The issue in both cases appears to be related to this JavaScript error: $ is not defined This is due to the compat component re-sync ( ICE-5665 ) being done after the work to hide Prototype functions ( ICE-5844 ). The re-sync reverted the Prototype hiding work which causes the JavaScript functions to not be found. I fixed up the MenuItemRenderer class but we there was overlap between the two different changes so we'll need to check to see if anything else got reverted.
              Hide
              Permalink
              Deryk Sinotte added a comment -

              Re-applied the Prototype hiding changes.

              Show
              Deryk Sinotte added a comment - Re-applied the Prototype hiding changes.
              Hide
              Permalink
              Mandeep Hayher added a comment -

              Tested successfully on Icefaces2 revision# 22076
              ICE-3856, ICE-3320 & ICE-1232 all passed in nightly regression test.

              Show
              Mandeep Hayher added a comment - Tested successfully on Icefaces2 revision# 22076 ICE-3856 , ICE-3320 & ICE-1232 all passed in nightly regression test.

                People

                • Assignee:
                  Deryk Sinotte
                  Reporter:
                  Mandeep Hayher
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: