ICEfaces
  1. ICEfaces
  2. ICE-7588

tooltip: ace:ajax "display" event doesn't function / displayListener doesn't function when combined with ace:ajax

    Details

    • Assignee Priority:
      P1

      Description

      As the title states...

      To reproduce load the test application and navigate to the 'Tooltip Ajax' test. There are separate tests for each problem.

        Activity

        Cruz Miraback created issue -
        Ken Fyten made changes -
        Field Original Value New Value
        Salesforce Case []
        Assignee Priority P1
        Assignee Nils Lundquist [ nils.lundquist ]
        Hide
        Nils Lundquist added a comment -

        This issue is due to ajax source defaulting to the first form on the page

        Show
        Nils Lundquist added a comment - This issue is due to ajax source defaulting to the first form on the page
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #26943 Mon Dec 19 13:13:00 MST 2011 nils.lundquist ICE-7588 - Tooltip not working because rendered tooltip script was not given a clientId. Thus javax.faces.source handling couldn't find the component clientId and was falling back to the first form on the page; this failed as decoding of listeners requires that source be correctly configured.
        Files Changed
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/tooltip/tooltip.js
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/tooltip/TooltipRenderer.java
        Hide
        Nils Lundquist added a comment -

        Fixed in r26943

        Tooltip script tag required a clientId, so that there is a clientId to find when JS handling for the request source goes looking for the source in the DOM.

        Show
        Nils Lundquist added a comment - Fixed in r26943 Tooltip script tag required a clientId, so that there is a clientId to find when JS handling for the request source goes looking for the source in the DOM.
        Nils Lundquist made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Fix Version/s 3.0.RC2 [ 10313 ]
        Hide
        yip.ng added a comment -
        Show
        yip.ng added a comment - ICE-7601
        Hide
        Cruz Miraback added a comment -

        Tested this fix just now using Icefaces3 rev# 27135 and it appears to be broken again. The ajaxListener isn't being called (or at the very least the message not being updated on the page).

        Show
        Cruz Miraback added a comment - Tested this fix just now using Icefaces3 rev# 27135 and it appears to be broken again. The ajaxListener isn't being called (or at the very least the message not being updated on the page).
        Cruz Miraback made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Hide
        Nils Lundquist added a comment -

        ICE-7601: add a containing span around the script so that Mojarra doesn't complain during update processing.
        Caused regression of ICE-7588.

        Show
        Nils Lundquist added a comment - ICE-7601 : add a containing span around the script so that Mojarra doesn't complain during update processing. Caused regression of ICE-7588 .
        Ken Fyten made changes -
        Salesforce Case []
        Fix Version/s 3.0.RC2 [ 10313 ]
        Affects Version/s 3.0.RC2 [ 10313 ]
        Affects Version/s 3.0 [ 10241 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #27155 Wed Jan 11 14:27:59 MST 2012 nils.lundquist Fixed Regression of ICE-7588 : ICE-7601 commit made by Deryk appended "_script" to clientId. ClientId must be explicit for correct ice.ace.ab ajax source resolution.
        Files Changed
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/tooltip/TooltipRenderer.java
        Hide
        Nils Lundquist added a comment -

        Revision #27155
        Committed by nils.lundquist
        Fixed Regression of ICE-7588 : ICE-7601 commit made by Deryk appended "_script" to clientId. ClientId must be explicit for correct ice.ace.ab ajax source resolution.

        Show
        Nils Lundquist added a comment - Revision #27155 Committed by nils.lundquist Fixed Regression of ICE-7588 : ICE-7601 commit made by Deryk appended "_script" to clientId. ClientId must be explicit for correct ice.ace.ab ajax source resolution.
        Nils Lundquist made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Carmen Cristurean added a comment -

        Re-tested fix with revision # 27166 in IE8, Firefox3.6, Chrome15, and it works as expected.

        Show
        Carmen Cristurean added a comment - Re-tested fix with revision # 27166 in IE8, Firefox3.6, Chrome15, and it works as expected.
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Nils Lundquist
            Reporter:
            Cruz Miraback
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: