ICEfaces
  1. ICEfaces
  2. ICE-5992

Allow arbitrary components in menuItem

    Details

    • Type: New Feature New Feature
    • Status: Closed
    • Priority: Major Major
    • Resolution: Invalid
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      icefaces2 compat
    • Affects:
      Documentation (User Guide, Ref. Guide, etc.), Sample App./Tutorial

      Description

      For menu items, that are not at the top level, and which are leaf nodes, meaning that they have no children, add ability to have arbitrary component inside the menu item. This will replace any rendering of the link, label, icon, arrows, etc. The arbitrary component will be specified by placing it within a facet named "content".

      There will likely be practical, implementation detail specific, limitations on which components may be placed within the menuItem. Most likely, components that themselves have popup behaviours will not work, and would require specific integration efforts to function.

        Activity

        Mark Collette created issue -
        Mark Collette made changes -
        Field Original Value New Value
        Salesforce Case []
        Fix Version/s 2.0-Beta2 [ 10242 ]
        Description For menu items, that are not at the top level, and which are leaf nodes, meaning that they have no children, add ability to have arbitrary component inside the menu item. This will replace any rendering of the link, label, icon, arrows, etc. The arbitrary component will be specified by placing it within a facet named "content".
        For menu items, that are not at the top level, and which are leaf nodes, meaning that they have no children, add ability to have arbitrary component inside the menu item. This will replace any rendering of the link, label, icon, arrows, etc. The arbitrary component will be specified by placing it within a facet named "content".

        There will likely be practical, implementation detail specific, limitations on which components may be placed within the menuItem. Most likely, components that themselves have popup behaviours will not work, and would require specific integration efforts to function.
        Assignee Mark Collette [ mark.collette ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #22172 Fri Aug 13 17:27:31 MDT 2010 mark.collette ICE-5992 : Allow arbitrary components in menuItem
        Files Changed
        Commit graph MODIFY /icefaces2/branches/icefaces-2.0.0-beta1.5/icefaces/compat/components/src/main/java/com/icesoft/faces/component/menubar/MenuItemRenderer.java
        Hide
        Mark Collette added a comment -

        Modified the MenuItemRenderer to make use of the content facet, and render that in place of the usual anchor, span, img tags. There is still a problem with the background colour styling not showing, and with clicks on the content component causing the menu to hide.

        Check-in of interim release of code from the morning.

        Subversion 22172

        Show
        Mark Collette added a comment - Modified the MenuItemRenderer to make use of the content facet, and render that in place of the usual anchor, span, img tags. There is still a problem with the background colour styling not showing, and with clicks on the content component causing the menu to hide. Check-in of interim release of code from the morning. Subversion 22172
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #22221 Sun Aug 22 19:45:54 MDT 2010 mark.collette ICE-5992 : Allow arbitrary components in menuItem
        Files Changed
        Commit graph MODIFY /icefaces2/branches/icefaces-2.0.0-beta1.5/icefaces/compat/core/src/main/resources/css/com/icesoft/faces/resources/css/royale/royale.css
        Commit graph MODIFY /icefaces2/branches/icefaces-2.0.0-beta1.5/icefaces/compat/components/src/main/java/com/icesoft/faces/component/CSS_DEFAULT.java
        Commit graph MODIFY /icefaces2/branches/icefaces-2.0.0-beta1.5/icefaces/compat/components/src/main/java/com/icesoft/faces/component/menubar/MenuItem.java
        Commit graph MODIFY /icefaces2/branches/icefaces-2.0.0-beta1.5/icefaces/compat/core/src/main/resources/css/com/icesoft/faces/resources/css/rime/rime-portlet.css
        Commit graph MODIFY /icefaces2/branches/icefaces-2.0.0-beta1.5/icefaces/compat/core/src/main/javascript/extras/extras.js
        Commit graph MODIFY /icefaces2/branches/icefaces-2.0.0-beta1.5/icefaces/compat/components/src/main/java/com/icesoft/faces/component/menubar/MenuBar.java
        Commit graph MODIFY /icefaces2/branches/icefaces-2.0.0-beta1.5/icefaces/compat/core/src/main/resources/css/com/icesoft/faces/resources/css/xp/xp.css
        Commit graph MODIFY /icefaces2/branches/icefaces-2.0.0-beta1.5/icefaces/compat/core/src/main/resources/css/com/icesoft/faces/resources/css/xp/xp-portlet.css
        Commit graph MODIFY /icefaces2/branches/icefaces-2.0.0-beta1.5/icefaces/compat/core/src/main/resources/css/com/icesoft/faces/resources/css/rime/rime.css
        Hide
        Mark Collette added a comment -

        Modified the javascript so that clicks within the content facet would not make the menus hide. To facilitate that, and to enable application styling, added a iceMnuItmCont style class, rendered on the div of menu items that have a content facet. This is as opposed to the regular iceMnuItm style class. Even before adding the new style class, keyboard navigation through content menu items did not work. With the style class, it broke even more. So, modified the menu keyboard navigation to support content menu items. Updated the themes to style iceMnuItmCont just like iceMnuItm.

        Subversion 22221

        Show
        Mark Collette added a comment - Modified the javascript so that clicks within the content facet would not make the menus hide. To facilitate that, and to enable application styling, added a iceMnuItmCont style class, rendered on the div of menu items that have a content facet. This is as opposed to the regular iceMnuItm style class. Even before adding the new style class, keyboard navigation through content menu items did not work. With the style class, it broke even more. So, modified the menu keyboard navigation to support content menu items. Updated the themes to style iceMnuItmCont just like iceMnuItm. Subversion 22221
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #22233 Wed Aug 25 18:48:31 MDT 2010 mark.collette ICE-5992 : Allow arbitrary components in menuItem
        Files Changed
        Commit graph MODIFY /icefaces2/branches/icefaces-2.0.0-beta1.5/icefaces/compat/components/src/main/java/com/icesoft/faces/component/menubar/MenuItemRenderer.java
        Commit graph MODIFY /icefaces2/branches/icefaces-2.0.0-beta1.5/icefaces/compat/core/src/main/resources/css/com/icesoft/faces/resources/css/rime/rime.css
        Hide
        Mark Collette added a comment -

        Accidentally committed a fix for ICE-5991 under this jira. This has become a catch-all for a bunch of issues, many of which are really pertaining to ICE-5991.
        Subversion 22233

        Show
        Mark Collette added a comment - Accidentally committed a fix for ICE-5991 under this jira. This has become a catch-all for a bunch of issues, many of which are really pertaining to ICE-5991 . Subversion 22233
        Ken Fyten made changes -
        Salesforce Case []
        Fix Version/s 2.0.0 [ 10230 ]
        Fix Version/s 1.8.2-EE-GA_P02 [ 10226 ]
        Fix Version/s 2.0-Beta2 [ 10242 ]
        Affects [Documentation (User Guide, Ref. Guide, etc.), Sample App./Tutorial]
        Affects Version/s 2.0-Beta1 [ 10231 ]
        Ken Fyten made changes -
        Salesforce Case []
        Assignee Priority P3
        Hide
        Mark Collette added a comment -

        The code for this exists on an ICEfaces 2 specific branch, so we might just do this for ICEfaces 2 and not 1.8.x.

        The outstanding issue for the menu code is to resolve the issue where we couldn't get menus to auto-size for IE6. There may be a way to apply the width hard-coding only to IE6, and let everything else auto-size. Then we could take this content facet code, sub-items arrow code and auto-sizing code, and commit it all together to trunk.

        Show
        Mark Collette added a comment - The code for this exists on an ICEfaces 2 specific branch, so we might just do this for ICEfaces 2 and not 1.8.x. The outstanding issue for the menu code is to resolve the issue where we couldn't get menus to auto-size for IE6. There may be a way to apply the width hard-coding only to IE6, and let everything else auto-size. Then we could take this content facet code, sub-items arrow code and auto-sizing code, and commit it all together to trunk.
        Ken Fyten made changes -
        Salesforce Case []
        Fix Version/s 1.8.2-EE-GA_P02 [ 10226 ]
        Ken Fyten made changes -
        Salesforce Case []
        Fix Version/s 2.1 [ 10241 ]
        Fix Version/s 2.0.0 [ 10230 ]
        Assignee Priority P3
        Arran Mccullough made changes -
        Salesforce Case [5007000000GDTsA]
        Ken Fyten made changes -
        Fix Version/s 3.1 [ 10312 ]
        Fix Version/s 3.0 [ 10241 ]
        Ken Fyten made changes -
        Fix Version/s 3.1 [ 10312 ]
        Assignee Mark Collette [ mark.collette ]
        Hide
        Ken Fyten added a comment -

        Marking as Closed / Invalid as part of legacy ICEfaces ICE / Compat component JIRA cleanup.

        Note: This issue may be resolved in a newer ICEfaces release, available here: http://www.icesoft.org/java/downloads/icefaces-downloads.jsf

        If the issue persists with the current ICEfaces release, please create a new JIRA for it.

        Show
        Ken Fyten added a comment - Marking as Closed / Invalid as part of legacy ICEfaces ICE / Compat component JIRA cleanup. Note: This issue may be resolved in a newer ICEfaces release, available here: http://www.icesoft.org/java/downloads/icefaces-downloads.jsf If the issue persists with the current ICEfaces release, please create a new JIRA for it.
        Ken Fyten made changes -
        Status Open [ 1 ] Closed [ 6 ]

          People

          • Assignee:
            Unassigned
            Reporter:
            Mark Collette
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: