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
              Mandeep Hayher created issue -
              Mandeep Hayher made changes -
              Field Original Value New Value
              Assignee Deryk Sinotte [ deryk.sinotte ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #22067 Fri Jul 30 16:08:31 MDT 2010 deryk.sinotte ICE-5945: re-applied the Prototype hiding changes
              Files Changed
              Commit graph MODIFY /icefaces2/trunk/icefaces/compat/components/src/main/java/com/icesoft/faces/component/menubar/MenuItemRenderer.java
              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.
              Deryk Sinotte made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Fix Version/s 2.0-Beta2 [ 10242 ]
              Resolution Fixed [ 1 ]
              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.
              Mandeep Hayher made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: