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

                Ascending order - Click to sort in descending order
                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.
                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
                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
                yip.ng added a comment -

                See ICE-4848.

                Show
                yip.ng added a comment - See ICE-4848 .
                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
                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.
                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.

                  People

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

                    Dates

                    • Created:
                      Updated:
                      Resolved: