ICEfaces
  1. ICEfaces
  2. ICE-3874

Add "displayOn" attribute on the ice:panelTooltip to support additional display triggering event types

          Details

          • Type: Improvement Improvement
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 1.7.2 SP1, 1.8DR#1
          • Fix Version/s: 1.8.2-RC1, 1.8.2
          • Component/s: ICE-Components
          • Labels:
            None
          • Environment:
            IE, FF
          • Affects:
            Documentation (User Guide, Ref. Guide, etc.), Sample App./Tutorial

            Description

            Hi ICEfaces team,

            It would be nice to add a displayOnClick attribute on the ice:panelTooltip as the same as menuPopup. Since the ice:panelTooltip often dynamically retrieve data from database, only when user needs to review some item details, then he/she clicks on the item to retrieve its data. Otherwise, if user don't want to review an item details, but he/she already his/her mouse on the item, and system will also retrieve this item data. This situation will waste some unnecessary database connections. I think that adding displayOnClick may prevent it.

            Thanks a lot for considering the jira in advance.

              Issue Links

              blocks

              Improvement - An improvement or enhancement to an existing feature or task. ICE-4848 Add "displayOn" demo to existing ice:panelTooltip demo in Component Showcase

              • Major - Major loss of function.
              • Closed

                Activity

                Descending order - Click to sort in ascending order
                Ken Fyten made changes -
                Status Resolved [ 5 ] Closed [ 6 ]
                Assignee Priority P3
                Repository Revision Date User Message
                ICEsoft Public SVN Repository #19910 Wed Dec 09 13:20:01 MST 2009 adnan.durrani 1.8.2EE panelTooltip changes moved to Compat panelTooltip towards ICE-4481. The changes include the fix of following JIRAs: ICE-3745, ICE-3874, ICE-4853, ICE-3874 and ICE-4952
                Files Changed
                Commit graph MODIFY /icefaces/scratchpads/glimmer/compat/core/src/main/resources/META-INF/resources/ice-extras.js
                Commit graph MODIFY /icefaces/scratchpads/glimmer/compat/component-metadata/src/main/resources/conf/ice_cust_properties/cust-paneltooltip-props.xml
                Commit graph MODIFY /icefaces/scratchpads/glimmer/compat/core/src/main/java/com/icesoft/faces/util/CoreUtils.java
                Ken Fyten made changes -
                Fix Version/s 1.8.2-RC1 [ 10210 ]
                yip.ng made changes -
                Status Reopened [ 4 ] Resolved [ 5 ]
                Resolution Fixed [ 1 ]
                Hide
                Permalink
                yip.ng added a comment -

                Done:

                <ice:panelTooltip id="provinceInfo" displayOn="dblclick" ...>

                See screenshot-2 for the TLD doc.

                Show
                yip.ng added a comment - Done: <ice:panelTooltip id="provinceInfo" displayOn="dblclick" ...> See screenshot-2 for the TLD doc.
                yip.ng made changes -
                Attachment screenshot-2.jpg [ 11939 ]
                Repository Revision Date User Message
                ICEsoft Public SVN Repository #19192 Tue Aug 25 13:06:10 MDT 2009 yip.ng ICE-3874: Added displayOn attribute for tooltip panel.
                Files Changed
                Commit graph MODIFY /icefaces/trunk/icefaces/component-metadata/src/main/resources/conf/ice_cust_properties/cust-paneltooltip-props.xml
                Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/util/CoreUtils.java
                Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/paneltooltip/PanelTooltip.java
                Commit graph MODIFY /icefaces/trunk/icefaces/bridge/lib/extras/tooltip_panelpopup.js
                Ken Fyten made changes -
                Summary Add a displayOnClick attribute on the ice:panelTooltip Add "displayOn" attribute on the ice:panelTooltip to support additional display triggering event types
                Hide
                Permalink
                Ken Fyten added a comment -

                After review of the displayOnClick functionality, it becomes obvious that what we really should have (instead) is a "displayOn" attribute that could accept the following types:

                "hover" (default)
                "click" - left mouse button clicked
                "altclick" - right mouse button clicked (tied to the "contextmenu" JS event)
                "dblclick" - double click

                If nothing is specified, "hover" is assumed.

                Show
                Ken Fyten added a comment - After review of the displayOnClick functionality, it becomes obvious that what we really should have (instead) is a "displayOn" attribute that could accept the following types: "hover" (default) "click" - left mouse button clicked "altclick" - right mouse button clicked (tied to the "contextmenu" JS event) "dblclick" - double click If nothing is specified, "hover" is assumed.
                Ken Fyten made changes -
                Resolution Fixed [ 1 ]
                Status Resolved [ 5 ] Reopened [ 4 ]
                Hide
                Permalink
                Ken Fyten added a comment -

                After reviewing the implemented feature, revising the spec.

                Show
                Ken Fyten added a comment - After reviewing the implemented feature, revising the spec.
                Hide
                Permalink
                yip.ng added a comment -

                See ICE-4848.

                Show
                yip.ng added a comment - See ICE-4848 .
                Hide
                Permalink
                Joanne Bai added a comment -

                Yip, Sorry that I forgot to mention one thing:

                displayOnClick works fine, but it does not actually "replace" mouse hover. In showcase sample, tooltips can be brought up by either mouse clicking or hover. Should they be coexistent? Or, when one of them is on, the other should be off? Thanks.

                Show
                Joanne Bai added a comment - Yip, Sorry that I forgot to mention one thing: displayOnClick works fine, but it does not actually "replace" mouse hover. In showcase sample, tooltips can be brought up by either mouse clicking or hover. Should they be coexistent? Or, when one of them is on, the other should be off? Thanks.
                Hide
                Permalink
                Joanne Bai added a comment -

                Verified successfully using component showcase on ICEfaces trunk 19177

                Tested on FF3.5 and IE7

                Show
                Joanne Bai added a comment - Verified successfully using component showcase on ICEfaces trunk 19177 Tested on FF3.5 and IE7
                Ken Fyten made changes -
                Link This issue blocks ICE-4848 [ ICE-4848 ]
                yip.ng made changes -
                Status Open [ 1 ] Resolved [ 5 ]
                Resolution Fixed [ 1 ]
                Hide
                Permalink
                yip.ng added a comment - - edited

                Attribute and functionality added:

                <ice:panelTooltip id="pnlSerPanelToolTip" displayOnClick="true" ...>

                See screenshot-1 for the TLD doc.

                Show
                yip.ng added a comment - - edited Attribute and functionality added: <ice:panelTooltip id="pnlSerPanelToolTip" displayOnClick="true" ...> See screenshot-1 for the TLD doc.
                yip.ng made changes -
                Attachment screenshot-1.jpg [ 11927 ]
                Repository Revision Date User Message
                ICEsoft Public SVN Repository #19174 Wed Aug 19 11:12:11 MDT 2009 yip.ng ICE-3874: Added displayOnClick attribute for panelTooltip.
                Files Changed
                Commit graph MODIFY /icefaces/trunk/icefaces/component-metadata/src/main/resources/conf/ice_cust_properties/cust-paneltooltip-props.xml
                Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/util/CoreUtils.java
                Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/paneltooltip/PanelTooltip.java
                Commit graph MODIFY /icefaces/trunk/icefaces/bridge/lib/extras/tooltip_panelpopup.js
                Ken Fyten made changes -
                Fix Version/s 1.8.2 [ 10190 ]
                Affects [Documentation (User Guide, Ref. Guide, etc.), Sample App./Tutorial]
                Assignee Priority P3
                Assignee Yip Ng [ yip.ng ]
                Sam Xiao made changes -
                Field Original Value New Value
                Salesforce Case [50070000007baph]
                Sheng Gu created issue -

                  People

                  • Assignee:
                    yip.ng
                    Reporter:
                    Sheng Gu
                  • Votes:
                    1 Vote for this issue
                    Watchers:
                    3 Start watching this issue

                    Dates

                    • Created:
                      Updated:
                      Resolved: