ICEfaces
  1. ICEfaces
  2. ICE-7362

ace:tooltip with displaylistener javascript error

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.1-Beta
    • Fix Version/s: 2.1-Beta2, 3.0
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      Tomcat 7.0.12
    • Assignee Priority:
      P1

      Description

      The poc has been updated to use the ace:tooltip with displayListener in Screen2. When hovering over the ace:tooltip the following javascript error shows up in the console:

      this.target.parents is not a function
      [Break On This Error] formId = this.target.parents('form:first').attr('id');

      In tooltip.js:

      ice.ace.Tooltip.prototype.triggerDisplayListener = function() {
      formId = this.target.parents('form:first').attr('id');

        Activity

        Hide
        yip.ng added a comment - - edited

        Display listener code added by us, not from PF or third-party?

        Show
        yip.ng added a comment - - edited Display listener code added by us, not from PF or third-party?
        Hide
        Brad Kroeger added a comment -

        I believe it was added by us in a previous JIRA improvement/enhancement

        Show
        Brad Kroeger added a comment - I believe it was added by us in a previous JIRA improvement/enhancement
        Hide
        yip.ng added a comment - - edited

        3rd-party tooltip plugin source code is a long string (22480) of garbled characters. Impossible to debug. See screenshot-01.png. See if we can fix problem without knowing the plugin source code.

        Show
        yip.ng added a comment - - edited 3rd-party tooltip plugin source code is a long string (22480) of garbled characters. Impossible to debug. See screenshot-01.png. See if we can fix problem without knowing the plugin source code.
        Hide
        yip.ng added a comment -

        Downloaded qTip 1.0.0-rc3. Copyright header is exactly the same, so we can safely use the uncompressed source code instead of the existing compressed source code.

        Also downloaded and used the qTip debug plugin. See screenshot-02.png for the debug mode in action.

        Error fixed. Bug is in our code added to PF code, not in qTip code.

        Revision: 26089


        Added : /icefaces-ee/scratchpads/grimlock/icefaces/ace/component/resources/icefaces.ace/tooltip/jquery.qtip-1.0.0-rc3.js
        Added : /icefaces-ee/scratchpads/grimlock/icefaces/ace/component/resources/icefaces.ace/tooltip/jquery.qtip-1.0.0-rc3.min.js
        Added : /icefaces-ee/scratchpads/grimlock/icefaces/ace/component/resources/icefaces.ace/tooltip/jquery.qtip.debug-1.0.0-rc3.js
        Modified : /icefaces-ee/scratchpads/grimlock/icefaces/ace/component/resources/icefaces.ace/tooltip/tooltip.js
        Modified : /icefaces-ee/scratchpads/grimlock/icefaces/ace/component/src/org/icefaces/ace/component/tooltip/TooltipMeta.java

        Show
        yip.ng added a comment - Downloaded qTip 1.0.0-rc3. Copyright header is exactly the same, so we can safely use the uncompressed source code instead of the existing compressed source code. Also downloaded and used the qTip debug plugin. See screenshot-02.png for the debug mode in action. Error fixed. Bug is in our code added to PF code, not in qTip code. Revision: 26089 Added : /icefaces-ee/scratchpads/grimlock/icefaces/ace/component/resources/icefaces.ace/tooltip/jquery.qtip-1.0.0-rc3.js Added : /icefaces-ee/scratchpads/grimlock/icefaces/ace/component/resources/icefaces.ace/tooltip/jquery.qtip-1.0.0-rc3.min.js Added : /icefaces-ee/scratchpads/grimlock/icefaces/ace/component/resources/icefaces.ace/tooltip/jquery.qtip.debug-1.0.0-rc3.js Modified : /icefaces-ee/scratchpads/grimlock/icefaces/ace/component/resources/icefaces.ace/tooltip/tooltip.js Modified : /icefaces-ee/scratchpads/grimlock/icefaces/ace/component/src/org/icefaces/ace/component/tooltip/TooltipMeta.java

          People

          • Assignee:
            yip.ng
            Reporter:
            Brad Kroeger
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: