You are viewing a plain text version of this content. The canonical link for it is here.
Posted to server-dev@james.apache.org by ma...@apache.org on 2010/01/29 12:14:49 UTC

svn commit: r904463 - /james/hupa/trunk/widgets/src/main/java/org/apache/hupa/widgets/ui/ToolTip.java

Author: manolo
Date: Fri Jan 29 11:14:46 2010
New Revision: 904463

URL: http://svn.apache.org/viewvc?rev=904463&view=rev
Log:
removes the instanceof checks from the org.apache.hupa.widgets.ui.ToolTip constructor and exposes them as generic constraints. Thanks to Robert Munteanu. HUPA-62

Modified:
    james/hupa/trunk/widgets/src/main/java/org/apache/hupa/widgets/ui/ToolTip.java

Modified: james/hupa/trunk/widgets/src/main/java/org/apache/hupa/widgets/ui/ToolTip.java
URL: http://svn.apache.org/viewvc/james/hupa/trunk/widgets/src/main/java/org/apache/hupa/widgets/ui/ToolTip.java?rev=904463&r1=904462&r2=904463&view=diff
==============================================================================
--- james/hupa/trunk/widgets/src/main/java/org/apache/hupa/widgets/ui/ToolTip.java (original)
+++ james/hupa/trunk/widgets/src/main/java/org/apache/hupa/widgets/ui/ToolTip.java Fri Jan 29 11:14:46 2010
@@ -54,19 +54,17 @@
         }
         
     };
-    public ToolTip(final Widget w) {
-        if (!(w instanceof HasMouseOverHandlers) || (!( w instanceof HasMouseOutHandlers)) || (!(w instanceof HasMouseMoveHandlers))) {
-            throw new IllegalArgumentException("Widget must implement HasMouseOverHandlers and HasMouseOutHandlers");
-        }
-        ((HasMouseOverHandlers) w).addMouseOverHandler(new MouseOverHandler() {
+
+    public <T extends Widget & HasMouseOverHandlers & HasMouseOutHandlers & HasMouseMoveHandlers> ToolTip(final T w) {
+        w.addMouseOverHandler(new MouseOverHandler() {
 
             public void onMouseOver(MouseOverEvent event) {
                 showTimer.schedule(1000);
             }
             
         });
-        
-        ((HasMouseOutHandlers)w).addMouseOutHandler(new MouseOutHandler() {
+
+        w.addMouseOutHandler(new MouseOutHandler() {
 
             public void onMouseOut(MouseOutEvent event) {
                 showTimer.cancel();
@@ -74,8 +72,8 @@
             }
             
         });
-        
-        ((HasMouseMoveHandlers) w).addMouseMoveHandler(new MouseMoveHandler() {
+
+        w.addMouseMoveHandler(new MouseMoveHandler() {
 
             public void onMouseMove(MouseMoveEvent event) {
                 y = event.getScreenY();



---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org