ICEfaces
  1. ICEfaces
  2. ICE-10179

ace:tooltip - showDelay issues when using ace:delegate

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: EE-3.3.0.GA_P02, 4.0
    • Fix Version/s: EE-3.3.0.GA_P05, 4.3
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      All

      Description

      When using the showDelay attribute of the ace:tooltip while it is used along with an ace:delegate/dataTable, the tooltip is displayed even if the mouse is moved away from the cell that the tooltip is set for.

      In a non-delegate sample, the mouse needs to be kept on the tooltip area for the entire time of the delay for it to be shown. If the mouse is moved away it is not shown.

        Activity

        Hide
        Arran Mccullough added a comment -

        Attached test case code and war file. Note, the icefaces core, ace, and compat jar files need to be added into the war for it to run.

        Steps:

        • Load welcomeICEfaces.jsf
        • Hover over each of the cell text in the second column.
        • Move the mouse away from the table.
        • After 5ish seconds, each tooltip will be shown in the order they were hovered over in.
        • There is also a few non delegate text's under the table. Hovering over one of these and moving the mouse away, the tooltip is not shown.
        Show
        Arran Mccullough added a comment - Attached test case code and war file. Note, the icefaces core, ace, and compat jar files need to be added into the war for it to run. Steps: Load welcomeICEfaces.jsf Hover over each of the cell text in the second column. Move the mouse away from the table. After 5ish seconds, each tooltip will be shown in the order they were hovered over in. There is also a few non delegate text's under the table. Hovering over one of these and moving the mouse away, the tooltip is not shown.
        Hide
        Arturo Zambrano added a comment - - edited

        r51819: fix to clear timeout function to display tooltip if the mouse stops hovering over the trigger element or if a different element triggers the tooltip display, in delegate mode
        r51820: committed fix to the EE 3.3 maintenance branch

        Show
        Arturo Zambrano added a comment - - edited r51819: fix to clear timeout function to display tooltip if the mouse stops hovering over the trigger element or if a different element triggers the tooltip display, in delegate mode r51820: committed fix to the EE 3.3 maintenance branch
        Hide
        Liana Munroe added a comment -

        Tested with attached test case, ICEfaces EE-3.3.0 maintenance branch and ICEfaces 4 trunk r51826, Tomcat 8, MS Edge, IE 11, 10, 9, 8, 7, FF 55, Chrome 60.
        For the most part this issue appears resolved however I found a possible bug where the tooltip is improperly triggered only when performing these specific steps (all browsers).

        1.) Use the attached test case with the lastest EE-3.3.0 maintenance branch libs.
        2.) In the test app hover in decending order, for one second, over each of the cells in the name column. Example: Hover over Volvo for one second, move directly down to Subaru for one second, then down to VW for one second.
        3.) Move the mouse away from the VW cell about halfway down the page in a southward motion. Leave the mouse stationary.
        4.) Wait ~5 seconds, the VW tooltip appears.

        This issue does not occur when moving the mouse east, west or north from the cell. Also it does not seem to occur if not hovering over each of the cells exactly as described in step 2.
        This issue was only occasionally seen with IF 4 libs, but is easily reproduced with EE-3.3.0 libs.

        Show
        Liana Munroe added a comment - Tested with attached test case, ICEfaces EE-3.3.0 maintenance branch and ICEfaces 4 trunk r51826, Tomcat 8, MS Edge, IE 11, 10, 9, 8, 7, FF 55, Chrome 60. For the most part this issue appears resolved however I found a possible bug where the tooltip is improperly triggered only when performing these specific steps (all browsers). 1.) Use the attached test case with the lastest EE-3.3.0 maintenance branch libs. 2.) In the test app hover in decending order, for one second, over each of the cells in the name column. Example: Hover over Volvo for one second, move directly down to Subaru for one second, then down to VW for one second. 3.) Move the mouse away from the VW cell about halfway down the page in a southward motion. Leave the mouse stationary. 4.) Wait ~5 seconds, the VW tooltip appears. This issue does not occur when moving the mouse east, west or north from the cell. Also it does not seem to occur if not hovering over each of the cells exactly as described in step 2. This issue was only occasionally seen with IF 4 libs, but is easily reproduced with EE-3.3.0 libs.
        Hide
        Arturo Zambrano added a comment -

        r51828, r51829: improved fix to cancel all tooltip displays when mousing out of target elements in delegate mode (3.3 EE maintenance branch and 4.0 trunk)

        Show
        Arturo Zambrano added a comment - r51828, r51829: improved fix to cancel all tooltip displays when mousing out of target elements in delegate mode (3.3 EE maintenance branch and 4.0 trunk)
        Hide
        Liana Munroe added a comment - - edited

        Verified using attached test app, ICEfaces 4 trunk, EE-3.3.0 maintenance branch r51830, Tomcat 8, MS Edge, IE 11, 10, 9, 8, 7, FF 54, Chrome 60.
        When testing on the EE-3.3.0 maintenance branch using attached test app on IE 10 and IE 7 there was a console error when first accessing the app. There seemed to be no impact on functionality.

        Message: Unable to get property 'instanceTag' of undefined or null reference
        Line: 118
        Char: 9
        Code: 0
        URI: http://10.18.39.102:8080/Case/faces/javax.faces.resource/icepush.uncompressed.js?ln=ice.push&v=3_3_0_170814
        
        Show
        Liana Munroe added a comment - - edited Verified using attached test app, ICEfaces 4 trunk, EE-3.3.0 maintenance branch r51830, Tomcat 8, MS Edge, IE 11, 10, 9, 8, 7, FF 54, Chrome 60. When testing on the EE-3.3.0 maintenance branch using attached test app on IE 10 and IE 7 there was a console error when first accessing the app. There seemed to be no impact on functionality. Message: Unable to get property 'instanceTag' of undefined or null reference Line: 118 Char: 9 Code: 0 URI: http: //10.18.39.102:8080/Case/faces/javax.faces.resource/icepush.uncompressed.js?ln=ice.push&v=3_3_0_170814
        Hide
        Arturo Zambrano added a comment -

        I haven't been able to reproduce the above issue. It doesn't seem related to these fixes. Was this happening before these fixes were committed? Does it happen if there is no icepush.jar in the lib folder?

        Show
        Arturo Zambrano added a comment - I haven't been able to reproduce the above issue. It doesn't seem related to these fixes. Was this happening before these fixes were committed? Does it happen if there is no icepush.jar in the lib folder?
        Hide
        Liana Munroe added a comment -

        Tested with ICEfaces EE-3.3.0 maintenance branch r51817. The console error occurs before the fixes were committed at r51819. It does not occur if icepush.jar is removed.

        Show
        Liana Munroe added a comment - Tested with ICEfaces EE-3.3.0 maintenance branch r51817. The console error occurs before the fixes were committed at r51819. It does not occur if icepush.jar is removed.
        Hide
        Liana Munroe added a comment -

        http://jira.icesoft.org/browse/ICE-11355 was created for console error.

        Show
        Liana Munroe added a comment - http://jira.icesoft.org/browse/ICE-11355 was created for console error.

          People

          • Assignee:
            Arturo Zambrano
            Reporter:
            Arran Mccullough
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: