ICEfaces
  1. ICEfaces
  2. ICE-7601

ace:tooltip not adopting dynamically updated attribute values

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.0.RC2
    • Fix Version/s: 3.0
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      ICEfaces 3.0 svn rvn #26993
    • Assignee Priority:
      P1

      Description

      The ace:tooltip no longer adopts attribute value changes that are made dynamically (after the component is initially rendered on the page).

      To reproduce, using the showcase Tooltip demo, change any of the effect configuration settings dynamically and a see that the tooltip will not adopt them unless you force a reload of the page.

      Analysis shows that the tooltip is being updated in the DOM update in response to the ajax request that changes the value, but the component doesn't update it's internal state.

      Here's the response that is returned when the showDelay is changed to 2 seconds (2000 mills). This looks to be correct, but the tooltip component doesn't respect the new attribute values:
       

      <?xml version='1.0' encoding='UTF-8'?>
      <partial-response><changes><update id="j_idt44:tooltipShowDelaySelector"><![CDATA[<select id="j_idt44:tooltipShowDelaySelector" name="j_idt44:tooltipShowDelaySelector" size="1"> <option value="500">0.5 seconds</option>
      <option value="1000">1 second</option>
      <option value="2000" selected="true">2 seconds</option>
      </select>]]></update><update id="j_idt44:tooltip"><![CDATA[<script id="j_idt44:tooltip" type="text/javascript">jQuery(function() {widget_j_idt44_tooltip = new ice.ace.Tooltip({"global":false,"id":"j_idt44:tooltip","displayListener":false,"forComponent":"j_idt44:text",content:document.getElementById('j_idt44:tooltip_content').innerHTML,"show":{"when":{"event":"mouseover"},"delay":2000,"effect":{"length":100,"type":"slide"}},"hide":{"when":{"event":"mouseout"},"delay":500,"effect":{"length":100,"type":"slide"}},"position":{"container":jQuery(ice.ace.escapeClientId('j_idt44:text')).parent(),"corner":{"target":"bottomRight","tooltip":"topLeft"}}});});</script>]]></update><update id="dynamic-code-compat"><![CDATA[<span id="dynamic-code-compat"></span>]]></update><update id="javax.faces.ViewState"><![CDATA[-6438305865883867682:-4979992529496162372]]></update><extension aceCallbackParam="validationFailed">{"validationFailed":false}</extension></changes></partial-response>
      1. screenshot-1.png
        190 kB
      2. screenshot-2.png
        175 kB
      3. screenshot-3.png
        250 kB
      4. screenshot-4.png
        287 kB
      5. screenshot-5.png
        288 kB

        Activity

        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
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #27068 Fri Dec 23 13:07:50 MST 2011 yip.ng ICE-7601: ace:tooltip not adopting dynamically updated attribute values.
        Files Changed
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/tooltip/tooltip.js
        Commit graph MODIFY /icefaces3/trunk/icefaces/samples/showcase/showcase/src/main/java/org/icefaces/samples/showcase/example/ace/tooltip/TooltipOverviewBean.java
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #27063 Thu Dec 22 16:02:22 MST 2011 deryk.sinotte ICE-7601: add a containing span around the script so that Mojarra doesn't complain during update processing
        Files Changed
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/tooltip/TooltipRenderer.java
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #27016 Wed Dec 21 12:25:10 MST 2011 ken.fyten Remove ace:tooltip demo configurability due to issues noted in ICE-7601.
        Files Changed
        Commit graph MODIFY /icefaces3/tags/icefaces-3.0.0.RC2/icefaces/samples/showcase/showcase/src/main/webapp/resources/examples/ace/tooltip/toolTip.xhtml

          People

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

            Dates

            • Created:
              Updated:
              Resolved: