ICEmobile
  1. ICEmobile
  2. MOBI-485

ContentMenuItem rendering 'parent' js argument which isn't used in layoutmenu.js

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.1 Final
    • Fix Version/s: 1.3.1
    • Component/s: Faces
    • Labels:
      None
    • Environment:
      n/a

      Description


      The ContentMenuItem renderer is rendering the argument 'parent' for the js call using the value of the contentNavBar. But this doesn't seemed to be used in the script. We should remove it if it's not needed.


      StringBuilder sb = new StringBuilder("mobi.layoutMenu.showContent('").append(stackClientId);
                          sb.append("', event");
                          sb.append(",{ selectedId: '").append(item.getValue()).append("'");
                          sb.append(",singleSubmit: ").append(item.isSingleSubmit());
                          sb.append(", parent: '").append(navBar.getClientId(facesContext)).append("'");

        Issue Links

          Activity

          Hide
          Judy Guglielmin added a comment -

          This component is getting totally overhauled for 1.4 Beta with use of mobi:transition and client side api for contentSTack.
          Once that has been completed (see link) this jira will no longer be relevant

          Show
          Judy Guglielmin added a comment - This component is getting totally overhauled for 1.4 Beta with use of mobi:transition and client side api for contentSTack. Once that has been completed (see link) this jira will no longer be relevant
          Hide
          Judy Guglielmin added a comment -

          once client side api is provided for contentStack, there will be no js for contentStackMenu, making this redundant

          Show
          Judy Guglielmin added a comment - once client side api is provided for contentStack, there will be no js for contentStackMenu, making this redundant
          Hide
          Philip Breau added a comment -

          Revision: 38344
          Author: philip.breau
          Date: October-02-13 8:49:25 PM
          Message:
          MOBI-485 ContentMenuItem rendering 'parent' js argument which isn't used in layoutmenu.js


          Modified : /icemobile/trunk/icemobile/jsf/components/component/src/org/icefaces/mobi/component/contentmenuitem/ContentMenuItemRenderer.java

          Show
          Philip Breau added a comment - Revision: 38344 Author: philip.breau Date: October-02-13 8:49:25 PM Message: MOBI-485 ContentMenuItem rendering 'parent' js argument which isn't used in layoutmenu.js Modified : /icemobile/trunk/icemobile/jsf/components/component/src/org/icefaces/mobi/component/contentmenuitem/ContentMenuItemRenderer.java

            People

            • Assignee:
              Philip Breau
              Reporter:
              Philip Breau
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: