ICEfaces
  1. ICEfaces
  2. ICE-5352

Compat MenuBar regression test cases fail

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0-Alpha2
    • Fix Version/s: 2.0-Alpha3, 2.0.0
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      N/A

      Description

      Menubar test cases do not work after migration. User can see the menus but clicking on the menu has no effect.... Can not access sub menus.
      Testcase can be found at :
      \repo\qa\trunk\Regression-Icefaces2\ICE-1232

      Only menuBar testcase that works after migration is ICE-3640 (menuItem used with attribute link)
      Testcase can be found at :
      \repo\qa\trunk\Regression-Icefaces2\ICE-3640


        Activity

        Mandeep Hayher created issue -
        Ken Fyten made changes -
        Field Original Value New Value
        Summary MenuBar test cases when migrated to Icefaces2 are not working Compat MenuBar regression test cases fail
        Salesforce Case []
        Fix Version/s 2.0-Beta [ 10032 ]
        Assignee Priority P2
        Assignee Yip Ng [ yip.ng ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #20618 Thu Feb 11 17:25:09 MST 2010 yip.ng ICE-5352: Fixed Compat MenuBar regression test cases failure.
        Files Changed
        Commit graph MODIFY /icefaces/scratchpads/glimmer/compat/core/src/main/javascript/extras/extras.js
        Commit graph MODIFY /icefaces/scratchpads/glimmer/compat/components/src/main/java/com/icesoft/faces/component/menubar/MenuBar.java
        Commit graph MODIFY /icefaces/scratchpads/glimmer/compat/components/src/main/java/com/icesoft/faces/component/menubar/MenuBarRenderer.java
        Hide
        yip.ng added a comment -

        Revision: 20618


        Modified : /icefaces/scratchpads/glimmer/compat/components/src/main/java/com/icesoft/faces/component/menubar/MenuBar.java
        Modified : /icefaces/scratchpads/glimmer/compat/components/src/main/java/com/icesoft/faces/component/menubar/MenuBarRenderer.java
        Modified : /icefaces/scratchpads/glimmer/compat/core/src/main/javascript/extras/extras.js

        Show
        yip.ng added a comment - Revision: 20618 Modified : /icefaces/scratchpads/glimmer/compat/components/src/main/java/com/icesoft/faces/component/menubar/MenuBar.java Modified : /icefaces/scratchpads/glimmer/compat/components/src/main/java/com/icesoft/faces/component/menubar/MenuBarRenderer.java Modified : /icefaces/scratchpads/glimmer/compat/core/src/main/javascript/extras/extras.js
        Hide
        yip.ng added a comment -

        Note that to make the test case behave exactly like in component showcase, you need to change some code in the test case. Changes made except adding action listeners and links for the menu items. Without listeners or links nothing would happen when menu items are clicked and the menu items wouldn't disappear. Component showcase would behave the same if you removed the listeners and links.

        Revision: 22437


        Modified : /qa/trunk/Regression-Icefaces2/ICE-1232/web/main.xhtml

        Show
        yip.ng added a comment - Note that to make the test case behave exactly like in component showcase, you need to change some code in the test case. Changes made except adding action listeners and links for the menu items. Without listeners or links nothing would happen when menu items are clicked and the menu items wouldn't disappear. Component showcase would behave the same if you removed the listeners and links. Revision: 22437 Modified : /qa/trunk/Regression-Icefaces2/ ICE-1232 /web/main.xhtml
        Hide
        yip.ng added a comment -
        Show
        yip.ng added a comment - Done. See the demo videos at: http://screencast.com/t/OGI5OWM4OD (FF) http://screencast.com/t/OWU1NTM2NGE (IE)
        yip.ng made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Fix Version/s 2.0.0 [ 10230 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Assignee Priority P2

          People

          • Assignee:
            yip.ng
            Reporter:
            Mandeep Hayher
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: