Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 287547 - Bad updating control tooltip
Summary: Bad updating control tooltip
Status: RESOLVED FIXED
Alias: None
Product: RAP
Classification: RT
Component: RWT (show other bugs)
Version: 1.2   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 1.3 M2   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard: sr122
Keywords:
Depends on:
Blocks: 288924
  Show dependency tree
 
Reported: 2009-08-25 08:34 EDT by NkD Mising name CLA
Modified: 2010-01-19 08:59 EST (History)
1 user (show)

See Also:

Attachments
Snippet fro reproduce problem (1.50 KB, text/plain)
2009-08-25 08:36 EDT, NkD Mising name CLA 		no flags Details
a solution (2.21 KB, patch)
2009-08-26 11:54 EDT, Tim Buschtoens CLA 		no flags Details | Diff
a better solution (7.64 KB, patch)
2009-08-28 07:06 EDT, Tim Buschtoens CLA 		ivan: iplog+
Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description NkD Mising name CLA 2009-08-25 08:34:59 EDT 
Reproduce problem (with snippet atachment):

1. Write something to Text2 and click to button
2. Text1 have right tooltip alogn Text2.
3. Write something else to Text2 and clisk to button.
4. Text1 have same tooltip as in point 2. (not update)
Comment 1 NkD Mising name CLA 2009-08-25 08:36:04 EDT 
Created attachment 145533 [details]
Snippet fro reproduce problem
Comment 2 NkD Mising name CLA 2009-08-26 01:45:57 EDT 
There seems to be problem with ToolTip singleton. 
I try rewrite WidgetManager.js. Append explicit call method _applyBoundToWidget and it's work for me, but I dont know about ToolTip in detail a maybe this fix is wrong:
....
setToolTip : function( widget, toolTipText ) {
      if( toolTipText != null && toolTipText != "" ) {
        widget.setUserData( "toolTipText", toolTipText );
        var tp = org.eclipse.rwt.widgets.ToolTip.getInstance();
        tp._applyBoundToWidget(widget, toolTipText);
        widget.setToolTip( tp );
      } else {
        this._removeToolTipPopup( widget );
      }
    },
...
Comment 3 Tim Buschtoens CLA 2009-08-26 11:54:32 EDT 
Created attachment 145695 [details]
a solution

The ToolTip doesn't update its content as long as it is bound to the same widget, i.e. no other widgets with ToolTips are hovered. It is solved in this patch by manually updating the content if needed.
Comment 4 Tim Buschtoens CLA 2009-08-27 09:52:49 EDT 
Comment on attachment 145695 [details]
a solution

The fix does not work in qooxdoo debug-mode.
Comment 5 Tim Buschtoens CLA 2009-08-28 07:06:03 EDT 
Created attachment 145924 [details]
a better solution

works better by checking the ToolTips parent first. Tests included.
Comment 6 Rüdiger Herrmann CLA 2009-09-08 05:34:23 EDT 
Applied patch to CVS HEAD
Comment 7 NkD Mising name CLA 2009-09-09 04:58:17 EDT 
Should you fix it in RAP 1.2.1 too ?
Comment 8 Rüdiger Herrmann CLA 2009-09-09 06:09:44 EDT 
(In reply to comment #7)
> Should you fix it in RAP 1.2.1 too ?
RAP 1.2.1 is already built and this fix does not justify the effort of a re-build. 
We will schedule it for 1.2.2 (see bug 288924).
Comment 9 Ivan Furnadjiev CLA 2010-01-19 08:59:17 EST 
Applied patch in v12_Maintenance.