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

          Activity

          Hide
          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
          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
          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
          yip.ng added a comment -

          See ICE-4848.

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