ICEfaces
  1. ICEfaces
  2. ICE-3946

Add displayListener to MenuPopup

          Details

          • Type: New Feature New Feature
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 1.7.2 SP1
          • Fix Version/s: 1.7.2-SP2, 1.8RC1, 1.8
          • Component/s: ICE-Components
          • Labels:
            None
          • Environment:
            All

            Description

            There is a dynamic MenuPopup where menu items are rendered according to the type of tree node clicked. Each type of tree node can have different menu items available and therefore need to be able to reference the calling node so that it can determine which menuItems to render. The current difficulty is that you cannot get a reference to the calling node unless it is clicked. The new feature request would be add a server round trip to the menuPopup component (ie. actionListener) which would be where the pre-processing of which menuItems to render would be done.

              Activity

              Ascending order - Click to sort in descending order
              Tyler Johnson created issue -
              Tyler Johnson made changes -
              Field Original Value New Value
              Salesforce Case [50070000007CEJi]
              Tyler Johnson made changes -
              Description There is a dynamic MenuPopup where menu items are rendered according to the type of tree node clicked. Each type of tree node can have different menu items available and therefore need to be able to reference the calling node so that it can determine which menuItems to render. The current difficulty is that you cannot get a reference to the calling node unless it is clicked. The new feature request would be add a server round trip to the menuPopup component (ie. actionListener) which would be where the pre-processing of which menuItems would be done.
              		There is a dynamic MenuPopup where menu items are rendered according to the type of tree node clicked. Each type of tree node can have different menu items available and therefore need to be able to reference the calling node so that it can determine which menuItems to render. The current difficulty is that you cannot get a reference to the calling node unless it is clicked. The new feature request would be add a server round trip to the menuPopup component (ie. actionListener) which would be where the pre-processing of which menuItems to render would be done.
              Ken Fyten made changes -
              Fix Version/s 1.8DR#3 [ 10143 ]
              Assignee Priority P1
              Assignee Mark Collette [ mark.collette ]
              Ken Fyten made changes -
              Assignee Mark Collette [ mark.collette ] Adnan Durrani [ adnan.durrani ]
              Adnan Durrani made changes -
              Status Open [ 1 ] In Progress [ 3 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #18198 Wed Jan 21 03:52:49 MST 2009 adnan.durrani Fix for ICE-3946(Add actionListener to MenuPopup)

              DisplayListener added to the MenuPopup component.
              Files Changed
              Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/menupopup/MenuPopup.java
              Commit graph MODIFY /icefaces/trunk/icefaces/component-metadata/src/main/resources/conf/ice_cust_properties/cust-menuPopup-props.xml
              Commit graph MODIFY /icefaces/trunk/icefaces/component-metadata/src/main/resources/conf/custom/menuPopup-renderer.xml
              Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/menupopup/MenuPopupRenderer.java
              Commit graph MODIFY /icefaces/trunk/icefaces/bridge/component/menu.js
              Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/menupopup/MenuPopupHelper.java
              Hide
              Permalink
              Adnan Durrani added a comment -

              Trunk revision: 18198

              The displayListener added to the MenuPopup component.

              By default the menuPopup doesn't cause any submit when displayed by the right click, that is why its contents can be static only but it could be dynamic by just registering the displayListener on it. The displayEvent gives the target component as well as the clientId of the target component in form of contextValue, which then can be use to produce dynamic contents.

              Show
              Adnan Durrani added a comment - Trunk revision: 18198 The displayListener added to the MenuPopup component. By default the menuPopup doesn't cause any submit when displayed by the right click, that is why its contents can be static only but it could be dynamic by just registering the displayListener on it. The displayEvent gives the target component as well as the clientId of the target component in form of contextValue, which then can be use to produce dynamic contents.
              Adnan Durrani made changes -
              Status In Progress [ 3 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Ken Fyten made changes -
              Fix Version/s 1.7.3 [ 10162 ]
              Ken Fyten made changes -
              Resolution Fixed [ 1 ]
              Status Resolved [ 5 ] Reopened [ 4 ]
              Ken Fyten made changes -
              Assignee Priority P1 P3
              Ken Fyten made changes -
              Assignee Priority P3 P1
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #18282 Tue Feb 03 13:20:13 MST 2009 adnan.durrani ICE-3946 fix back ported to 1.7
              Files Changed
              Commit graph MODIFY /icefaces/branches/icefaces-1.7/icefaces/component-metadata/src/main/resources/conf/ice_cust_properties/cust-menuPopup-props.xml
              Commit graph MODIFY /icefaces/branches/icefaces-1.7/icefaces/component/src/com/icesoft/faces/component/menupopup/MenuPopup.java
              Commit graph MODIFY /icefaces/branches/icefaces-1.7/icefaces/bridge/component/menu.js
              Commit graph MODIFY /icefaces/branches/icefaces-1.7/icefaces/component/src/com/icesoft/faces/component/menupopup/MenuPopupHelper.java
              Commit graph MODIFY /icefaces/branches/icefaces-1.7/icefaces/component-metadata/src/main/resources/conf/custom/menuPopup-component.xml
              Commit graph MODIFY /icefaces/branches/icefaces-1.7/icefaces/component/src/com/icesoft/faces/component/menupopup/MenuPopupRenderer.java
              Hide
              Permalink
              Adnan Durrani added a comment -

              Modified: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\bridge\component\menu.js
              Modified: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component\src\com\icesoft\faces\component\menupopup\MenuPopup.java
              Modified: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component\src\com\icesoft\faces\component\menupopup\MenuPopupHelper.java
              Modified: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component\src\com\icesoft\faces\component\menupopup\MenuPopupRenderer.java
              Modified: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component-metadata\src\main\resources\conf\custom\menuPopup-component.xml
              Modified: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component-metadata\src\main\resources\conf\ice_cust_properties\cust-menuPopup-props.xml
              Sending content: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component\src\com\icesoft\faces\component\menupopup\MenuPopup.java
              Sending content: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component\src\com\icesoft\faces\component\menupopup\MenuPopupHelper.java
              Sending content: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component-metadata\src\main\resources\conf\ice_cust_properties\cust-menuPopup-props.xml
              Sending content: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component-metadata\src\main\resources\conf\custom\menuPopup-component.xml
              Sending content: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\bridge\component\menu.js
              Sending content: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component\src\com\icesoft\faces\component\menupopup\MenuPopupRenderer.java
              Completed: At revision: 18282

              Show
              Adnan Durrani added a comment - Modified: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\bridge\component\menu.js Modified: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component\src\com\icesoft\faces\component\menupopup\MenuPopup.java Modified: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component\src\com\icesoft\faces\component\menupopup\MenuPopupHelper.java Modified: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component\src\com\icesoft\faces\component\menupopup\MenuPopupRenderer.java Modified: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component-metadata\src\main\resources\conf\custom\menuPopup-component.xml Modified: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component-metadata\src\main\resources\conf\ice_cust_properties\cust-menuPopup-props.xml Sending content: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component\src\com\icesoft\faces\component\menupopup\MenuPopup.java Sending content: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component\src\com\icesoft\faces\component\menupopup\MenuPopupHelper.java Sending content: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component-metadata\src\main\resources\conf\ice_cust_properties\cust-menuPopup-props.xml Sending content: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component-metadata\src\main\resources\conf\custom\menuPopup-component.xml Sending content: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\bridge\component\menu.js Sending content: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component\src\com\icesoft\faces\component\menupopup\MenuPopupRenderer.java Completed: At revision: 18282
              Adnan Durrani made changes -
              Status Reopened [ 4 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Hide
              Permalink
              Arran Mccullough added a comment -

              There seems to be an issue with using Facelets. When the Facelets view-handler and jar file is added the displayListener is not called. On right click of the panelGroup. The menuPopup is shown but the display event is not fired.

              Show
              Arran Mccullough added a comment - There seems to be an issue with using Facelets. When the Facelets view-handler and jar file is added the displayListener is not called. On right click of the panelGroup. The menuPopup is shown but the display event is not fired.
              Hide
              Permalink
              Arran Mccullough added a comment -

              Test case for issue with Facelets. Built to be deployed on Tomcat 6.0.

              Show
              Arran Mccullough added a comment - Test case for issue with Facelets. Built to be deployed on Tomcat 6.0.
              Arran Mccullough made changes -
              Attachment Case5099Example.war [ 11510 ]
              Arran Mccullough made changes -
              Resolution Fixed [ 1 ]
              Status Resolved [ 5 ] Reopened [ 4 ]
              Ken Fyten made changes -
              Security Private [ 10001 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #18333 Wed Feb 11 14:15:54 MST 2009 mark.collette ICE-3946 : Add displayListener to MenuPopup
              Files Changed
              Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/facelets/IceComponentHandler.java
              Ken Fyten made changes -
              Comment [ A comment with security level 'icesoft-internal-developers' was removed. ]
              Mark Collette made changes -
              Summary Add actionListener to MenuPopup Add displayListener to MenuPopup
              Hide
              Permalink
              Mark Collette added a comment -

              Added support for Facelets to handle the displayListener property on menuPopup.

              Subversion 18333
              icefaces\component\src\com\icesoft\faces\component\facelets\IceComponentHandler.java

              Show
              Mark Collette added a comment - Added support for Facelets to handle the displayListener property on menuPopup. Subversion 18333 icefaces\component\src\com\icesoft\faces\component\facelets\IceComponentHandler.java
              Mark Collette made changes -
              Status Reopened [ 4 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #18345 Thu Feb 12 17:10:49 MST 2009 adnan.durrani reverting back the changes made to revision 18282[The ICE-3946 fix does not need to be back ported]
              Files Changed
              Commit graph MODIFY /icefaces/branches/icefaces-1.7/icefaces/component-metadata/src/main/resources/conf/ice_cust_properties/cust-menuPopup-props.xml
              Commit graph MODIFY /icefaces/branches/icefaces-1.7/icefaces/component/src/com/icesoft/faces/component/menupopup/MenuPopup.java
              Commit graph MODIFY /icefaces/branches/icefaces-1.7/icefaces/bridge/component/menu.js
              Commit graph MODIFY /icefaces/branches/icefaces-1.7/icefaces/component/src/com/icesoft/faces/component/menupopup/MenuPopupHelper.java
              Commit graph MODIFY /icefaces/branches/icefaces-1.7/icefaces/component-metadata/src/main/resources/conf/custom/menuPopup-component.xml
              Commit graph MODIFY /icefaces/branches/icefaces-1.7/icefaces/component/src/com/icesoft/faces/component/menupopup/MenuPopupRenderer.java
              Ken Fyten made changes -
              Assignee Priority P1
              Affects Version/s 1.7.2 SP1 [ 10144 ]
              Affects Version/s 1.8DR#2 [ 10142 ]
              Security Private [ 10001 ]
              Ken Fyten made changes -
              Attachment Case5099Example.war [ 11510 ]
              Hide
              Permalink
              Adnan Durrani added a comment -

              Changes reverted from the branch 1.7

              Modified: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\bridge\component\menu.js
              Modified: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component\src\com\icesoft\faces\component\menupopup\MenuPopup.java
              Modified: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component\src\com\icesoft\faces\component\menupopup\MenuPopupHelper.java
              Modified: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component\src\com\icesoft\faces\component\menupopup\MenuPopupRenderer.java
              Modified: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component-metadata\src\main\resources\conf\custom\menuPopup-component.xml
              Modified: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component-metadata\src\main\resources\conf\ice_cust_properties\cust-menuPopup-props.xml
              Sending content: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component\src\com\icesoft\faces\component\menupopup\MenuPopup.java
              Sending content: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component\src\com\icesoft\faces\component\menupopup\MenuPopupHelper.java
              Sending content: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component-metadata\src\main\resources\conf\ice_cust_properties\cust-menuPopup-props.xml
              Sending content: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component-metadata\src\main\resources\conf\custom\menuPopup-component.xml
              Sending content: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\bridge\component\menu.js
              Sending content: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component\src\com\icesoft\faces\component\menupopup\MenuPopupRenderer.java
              Completed: At revision: 18345

              Show
              Adnan Durrani added a comment - Changes reverted from the branch 1.7 Modified: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\bridge\component\menu.js Modified: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component\src\com\icesoft\faces\component\menupopup\MenuPopup.java Modified: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component\src\com\icesoft\faces\component\menupopup\MenuPopupHelper.java Modified: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component\src\com\icesoft\faces\component\menupopup\MenuPopupRenderer.java Modified: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component-metadata\src\main\resources\conf\custom\menuPopup-component.xml Modified: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component-metadata\src\main\resources\conf\ice_cust_properties\cust-menuPopup-props.xml Sending content: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component\src\com\icesoft\faces\component\menupopup\MenuPopup.java Sending content: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component\src\com\icesoft\faces\component\menupopup\MenuPopupHelper.java Sending content: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component-metadata\src\main\resources\conf\ice_cust_properties\cust-menuPopup-props.xml Sending content: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component-metadata\src\main\resources\conf\custom\menuPopup-component.xml Sending content: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\bridge\component\menu.js Sending content: D:\work\development\freshcopy\svn\ossrepo\icefaces\branches\icefaces-1.7\icefaces\component\src\com\icesoft\faces\component\menupopup\MenuPopupRenderer.java Completed: At revision: 18345
              Ken Fyten made changes -
              Fix Version/s 1.8 [ 10161 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]
              Assignee Adnan Durrani [ adnan.durrani ]

                People

                • Assignee:
                  Unassigned
                  Reporter:
                  Tyler Johnson
                • Votes:
                  1 Vote for this issue
                  Watchers:
                  1 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: