From 1bf9cba4c6282163fa56ef7f9780ed0e2268e486 Mon Sep 17 00:00:00 2001 From: John Alhroos Date: Tue, 16 Aug 2011 12:20:30 +0000 Subject: [PATCH] Fixed issue with toggling tooltips on/off #5414 svn changeset:20417/svn branch:6.7 --- src/com/vaadin/terminal/gwt/client/ui/VScrollTable.java | 8 +++++++- src/com/vaadin/ui/Table.java | 6 ++++-- 2 files changed, 11 insertions(+), 3 deletions(-) diff --git a/src/com/vaadin/terminal/gwt/client/ui/VScrollTable.java b/src/com/vaadin/terminal/gwt/client/ui/VScrollTable.java index 1f854f18f8..ad69d8a751 100644 --- a/src/com/vaadin/terminal/gwt/client/ui/VScrollTable.java +++ b/src/com/vaadin/terminal/gwt/client/ui/VScrollTable.java @@ -4426,6 +4426,9 @@ public class VScrollTable extends FlowPanel implements Table, ScrollHandler, if (rowDescription != null && !rowDescription.equals("")) { TooltipInfo info = new TooltipInfo(rowDescription); client.registerTooltip(VScrollTable.this, rowElement, info); + } else { + // Remove possibly previously set tooltip + client.registerTooltip(VScrollTable.this, rowElement, null); } tHead.getColumnAlignments(); @@ -4526,9 +4529,12 @@ public class VScrollTable extends FlowPanel implements Table, ScrollHandler, } } - if (description != null && client != null) { + if (description != null && !description.equals("")) { TooltipInfo info = new TooltipInfo(description); client.registerTooltip(VScrollTable.this, td, info); + } else { + // Remove possibly previously set tooltip + client.registerTooltip(VScrollTable.this, td, null); } td.appendChild(container); diff --git a/src/com/vaadin/ui/Table.java b/src/com/vaadin/ui/Table.java index d9446de5d1..fd18bf0fd7 100644 --- a/src/com/vaadin/ui/Table.java +++ b/src/com/vaadin/ui/Table.java @@ -46,7 +46,6 @@ import com.vaadin.terminal.Resource; import com.vaadin.terminal.gwt.client.MouseEventDetails; import com.vaadin.terminal.gwt.client.ui.VScrollTable; import com.vaadin.terminal.gwt.client.ui.dd.VLazyInitItemIdentifiers; -import com.vaadin.ui.AbstractSelect.ItemDescriptionGenerator; /** *

@@ -4586,7 +4585,10 @@ public class Table extends AbstractSelect implements Action.Container, * The generator to use or null to disable */ public void setItemDescriptionGenerator(ItemDescriptionGenerator generator){ - itemDescriptionGenerator = generator; + if (generator != itemDescriptionGenerator) { + itemDescriptionGenerator = generator; + refreshRenderedCells(); + } } /** -- 2.39.5