Add setToolTipText method to MenuItem

[eclipse-rap]

Add setToolTipText method to MenuItem

[ifurnadjiev]

Probably adding the method is enough. As seen from the code the toolTip text is set on the menu shell. Shell#setToolTipText is already available.

[phermsdorf]

Compile-wise this works, but no Tooltip shows up when hovering over the MenuItems. (which works in SWT)
Should I anyway make a PR to integrate that as first step?

[ifurnadjiev]

I've double checked the SWT implementation and it is different at least on Windows:

public void setToolTipText (String toolTip) {
	checkWidget ();
	if (toolTip == null && itemToolTip != null) {
		itemToolTip.setVisible (false);
		itemToolTip = null;
	}
	if (toolTip == null || toolTip.trim().length() == 0
			|| (itemToolTip != null && toolTip.equals(itemToolTip.getMessage()))) return;
	itemToolTip = new MenuItemToolTip (this.getParent().getShell());
	itemToolTip.setMessage (toolTip);
	itemToolTip.setVisible (false);
}

There is an internal class MenuItemToolTip extends ToolTip that handles the MenuItem tooltip.
The tooltip visibility is controlled by the method

void showTooltip (int x, int y) {
	if (itemToolTip == null) return;
	itemToolTip.setLocationInPixels (x, y);
	itemToolTip.setVisible (true);
}

which is called by the Menu class.

[ifurnadjiev]

Here it is: https://github.com/eclipse-platform/eclipse.platform.swt/blob/master/bundles/org.eclipse.swt/Eclipse%20SWT/win32/org/eclipse/swt/widgets/MenuItem.java#L1070

[ifurnadjiev]

Maybe this is the way to go in RWT as well.

[ifurnadjiev]

I'll try to come with complete JS implementation here, similar to the other widgets.

[phermsdorf]

I tried to adapt that, but the tooltip does not show up when hovering over the MenuItem's.
I think "nobody" handles the hovering and calling getTooltipText().
I tried to adapt it like eg. ToolItems tooltip handling:

• I added WidgetLCAUtil.preserveToolTipText( item, item.getToolTipText() ) in MenuItemLCA like it's done in ToolItemLCA
• and added case "rwt.widgets.MenuItem": in

  org.eclipse.rap/bundles/org.eclipse.rap.rwt/js/rwt/widgets/util/ToolTipConfig.js

  Lines 18 to 39 in eb49f00

  	getConfig : function( widget ) {
  	if( widget.getParent() instanceof rwt.widgets.CoolBar ) {
  	return this._getClickableConfig( widget );
  	}
  	switch( widget.classname ) {
  	case "rwt.widgets.ControlDecorator":
  	return this._quickConfig;
  	case "rwt.widgets.Label":
  	return this._getLabelConfig( widget );
  	case "rwt.widgets.Button":
  	case "rwt.widgets.TabItem":
  	case "rwt.widgets.CTabItem":
  	return this._getClickableConfig( widget );
  	case "rwt.widgets.Text":
  	case "rwt.widgets.Spinner":
  	case "rwt.widgets.Combo":
  	case "rwt.widgets.DateTimeDate":
  	case "rwt.widgets.DateTimeTime":
  	return this._fieldConfig;
  	case "rwt.widgets.ToolItem":
  	return this._getClickableConfig( widget );
  	case "rwt.widgets.ProgressBar":

  
  like the ToolItem but it still does not work
  (not that I'm really knowing what I'm doing here but I just tried to do the same like ToolItem does ...)

Any ideas?

[ifurnadjiev]

@phermsdorf I've created a fully working implementation and a pull request. What is probably missing for you are the changes in MenuItemHandler.js.