Fixed #2853 - No tooltip for embedded content

Added tooltip for embedded containing an image

svn changeset:7943/svn branch:6.0

2 files changed, 10 insertions, 1 deletions

diff --git a/src/com/vaadin/terminal/gwt/client/ui/VEmbedded.java b/src/com/vaadin/terminal/gwt/client/ui/VEmbedded.java
index b946896ed3..c15c738a29 100644
--- a/src/com/vaadin/terminal/gwt/client/ui/VEmbedded.java
+++ b/src/com/vaadin/terminal/gwt/client/ui/VEmbedded.java
@@ -19,6 +19,7 @@ import com.vaadin.terminal.gwt.client.ApplicationConnection;
 import com.vaadin.terminal.gwt.client.Paintable;
 import com.vaadin.terminal.gwt.client.UIDL;
 import com.vaadin.terminal.gwt.client.Util;
+import com.vaadin.terminal.gwt.client.VTooltip;
 
 public class VEmbedded extends HTML implements Paintable {
     private static String CLASSNAME = "v-embedded";
@@ -85,6 +86,12 @@ public class VEmbedded extends HTML implements Paintable {
                     getElement().appendChild(el);
                 }
 
+                /*
+                 * Sink tooltip events so tooltip is displayed when hovering the
+                 * image.
+                 */
+                sinkEvents(VTooltip.TOOLTIP_EVENTS);
+
             } else if (type.equals("browser")) {
                 if (browserElement == null) {
                     setHTML("<iframe width=\"100%\" height=\"100%\" frameborder=\"0\" src=\""
@@ -221,5 +228,7 @@ public class VEmbedded extends HTML implements Paintable {
         if (DOM.eventGetType(event) == Event.ONLOAD) {
             Util.notifyParentOfSizeChange(this, true);
         }
+
+        client.handleTooltipEvent(event, this);
     }
 }
diff --git a/src/com/vaadin/tests/components/embedded/EmbeddedTooltip.java b/src/com/vaadin/tests/components/embedded/EmbeddedTooltip.java
index 6ce788f262..37dbacb830 100644
--- a/src/com/vaadin/tests/components/embedded/EmbeddedTooltip.java
+++ b/src/com/vaadin/tests/components/embedded/EmbeddedTooltip.java
@@ -19,7 +19,7 @@ public class EmbeddedTooltip extends TestBase {
     @Override

     protected void setup() {

         Embedded e = new Embedded("Embedded caption", new ThemeResource(

-                "icons/64/ok.png"));

+                "../runo/icons/64/ok.png"));

         e

                 .setDescription("Embedded tooltip, only shown on caption, not on the image");

         addComponent(e);