#3012 Remove some deprecated GWT listener use (focus and loading related)

svn changeset:8588/svn branch:6.1

diff --git a/src/com/vaadin/terminal/gwt/client/ui/VFilterSelect.java b/src/com/vaadin/terminal/gwt/client/ui/VFilterSelect.java
index bdea4fbbe792a7f5de929551444c63a35dea6e17..56bf69e25adbf1a2cb62b276c27bd9e57997f2ba 100644 (file)
--- a/src/com/vaadin/terminal/gwt/client/ui/VFilterSelect.java
+++ b/src/com/vaadin/terminal/gwt/client/ui/VFilterSelect.java
@@ -10,13 +10,19 @@ import java.util.Date;
 import java.util.Iterator;
 import java.util.List;
 
+import com.google.gwt.event.dom.client.BlurEvent;
+import com.google.gwt.event.dom.client.BlurHandler;
 import com.google.gwt.event.dom.client.ClickEvent;
 import com.google.gwt.event.dom.client.ClickHandler;
+import com.google.gwt.event.dom.client.FocusEvent;
+import com.google.gwt.event.dom.client.FocusHandler;
 import com.google.gwt.event.dom.client.KeyCodes;
 import com.google.gwt.event.dom.client.KeyDownEvent;
 import com.google.gwt.event.dom.client.KeyDownHandler;
 import com.google.gwt.event.dom.client.KeyUpEvent;
 import com.google.gwt.event.dom.client.KeyUpHandler;
+import com.google.gwt.event.dom.client.LoadEvent;
+import com.google.gwt.event.dom.client.LoadHandler;
 import com.google.gwt.user.client.Command;
 import com.google.gwt.user.client.DOM;
 import com.google.gwt.user.client.Element;
@@ -24,14 +30,11 @@ import com.google.gwt.user.client.Event;
 import com.google.gwt.user.client.Window;
 import com.google.gwt.user.client.ui.Composite;
 import com.google.gwt.user.client.ui.FlowPanel;
-import com.google.gwt.user.client.ui.FocusListener;
 import com.google.gwt.user.client.ui.HTML;
 import com.google.gwt.user.client.ui.Image;
-import com.google.gwt.user.client.ui.LoadListener;
 import com.google.gwt.user.client.ui.PopupListener;
 import com.google.gwt.user.client.ui.PopupPanel;
 import com.google.gwt.user.client.ui.TextBox;
-import com.google.gwt.user.client.ui.Widget;
 import com.google.gwt.user.client.ui.PopupPanel.PositionCallback;
 import com.google.gwt.user.client.ui.SuggestOracle.Suggestion;
 import com.vaadin.terminal.gwt.client.ApplicationConnection;
@@ -47,7 +50,8 @@ import com.vaadin.terminal.gwt.client.VTooltip;
  * TODO needs major refactoring (to be extensible etc)
  */
 public class VFilterSelect extends Composite implements Paintable, Field,
-        KeyDownHandler, KeyUpHandler, ClickHandler, FocusListener, Focusable {
+        KeyDownHandler, KeyUpHandler, ClickHandler, FocusHandler, BlurHandler,
+        Focusable {
 
     public class FilterSelectSuggestion implements Suggestion, Command {
 
@@ -589,11 +593,8 @@ public class VFilterSelect extends Composite implements Paintable, Field,
 
     public VFilterSelect() {
         selectedItemIcon.setStyleName("v-icon");
-        selectedItemIcon.addLoadListener(new LoadListener() {
-            public void onError(Widget sender) {
-            }
-
-            public void onLoad(Widget sender) {
+        selectedItemIcon.addLoadHandler(new LoadHandler() {
+            public void onLoad(LoadEvent event) {
                 updateRootWidth();
                 updateSelectedIconPosition();
             }
@@ -607,7 +608,8 @@ public class VFilterSelect extends Composite implements Paintable, Field,
         tb.addKeyDownHandler(this);
         tb.addKeyUpHandler(this);
         tb.setStyleName(CLASSNAME + "-input");
-        tb.addFocusListener(this);
+        tb.addFocusHandler(this);
+        tb.addBlurHandler(this);
         popupOpener.setStyleName(CLASSNAME + "-button");
         popupOpener.addClickHandler(this);
     }
@@ -987,7 +989,7 @@ public class VFilterSelect extends Composite implements Paintable, Field,
         return w;
     }-*/;
 
-    public void onFocus(Widget sender) {
+    public void onFocus(FocusEvent event) {
         focused = true;
         if (prompting) {
             setPromptingOff("");
@@ -995,7 +997,7 @@ public class VFilterSelect extends Composite implements Paintable, Field,
         addStyleDependentName("focus");
     }
 
-    public void onLostFocus(Widget sender) {
+    public void onBlur(BlurEvent event) {
         focused = false;
         if (!suggestionPopup.isAttached() || suggestionPopup.isJustClosed()) {
             // typing so fast the popup was never opened, or it's just closed

diff --git a/src/com/vaadin/terminal/gwt/client/ui/VOptionGroup.java b/src/com/vaadin/terminal/gwt/client/ui/VOptionGroup.java
index 2f76ee1bef68a5343321c06b6bba9d692aff0303..3565175c898baba08f05d28537b53a290737b8fd 100644 (file)
--- a/src/com/vaadin/terminal/gwt/client/ui/VOptionGroup.java
+++ b/src/com/vaadin/terminal/gwt/client/ui/VOptionGroup.java
@@ -10,7 +10,7 @@ import java.util.Map;
 \r
 import com.google.gwt.event.dom.client.ClickEvent;\r
 import com.google.gwt.user.client.ui.CheckBox;\r
-import com.google.gwt.user.client.ui.HasFocus;\r
+import com.google.gwt.user.client.ui.Focusable;\r
 import com.google.gwt.user.client.ui.Panel;\r
 import com.google.gwt.user.client.ui.RadioButton;\r
 import com.google.gwt.user.client.ui.Widget;\r
@@ -96,7 +96,7 @@ public class VOptionGroup extends VOptionGroupBase {
     public void focus() {\r
         Iterator<Widget> iterator = panel.iterator();\r
         if (iterator.hasNext()) {\r
-            ((HasFocus) iterator.next()).setFocus(true);\r
+            ((Focusable) iterator.next()).setFocus(true);\r
         }\r
     }\r
 \r

diff --git a/src/com/vaadin/terminal/gwt/client/ui/VPopupView.java b/src/com/vaadin/terminal/gwt/client/ui/VPopupView.java
index 23fe10f3717220f12537c860edcf583ad8b48aa2..69acd0962dc45882ccce23e688acc8f7fac78b5f 100644 (file)
--- a/src/com/vaadin/terminal/gwt/client/ui/VPopupView.java
+++ b/src/com/vaadin/terminal/gwt/client/ui/VPopupView.java
@@ -8,8 +8,8 @@ import com.google.gwt.event.dom.client.ClickHandler;
 import com.google.gwt.user.client.DOM;
 import com.google.gwt.user.client.Element;
 import com.google.gwt.user.client.Event;
+import com.google.gwt.user.client.ui.Focusable;
 import com.google.gwt.user.client.ui.HTML;
-import com.google.gwt.user.client.ui.HasFocus;
 import com.google.gwt.user.client.ui.Label;
 import com.google.gwt.user.client.ui.PopupListener;
 import com.google.gwt.user.client.ui.PopupPanel;
@@ -268,8 +268,8 @@ public class VPopupView extends HTML implements Container {
          */
         public void syncChildren() {
             // Notify children with focus
-            if ((popupComponentWidget instanceof HasFocus)) {
-                ((HasFocus) popupComponentWidget).setFocus(false);
+            if ((popupComponentWidget instanceof Focusable)) {
+                ((Focusable) popupComponentWidget).setFocus(false);
             }
 
             // Notify children that have used the keyboard

diff --git a/src/com/vaadin/terminal/gwt/client/ui/VTextField.java b/src/com/vaadin/terminal/gwt/client/ui/VTextField.java
index f2be96438dfae3bdcc0035157c4a99a89f736794..9d3321c84afbc79a307990c8d1e924b63d01e0d7 100644 (file)
--- a/src/com/vaadin/terminal/gwt/client/ui/VTextField.java
+++ b/src/com/vaadin/terminal/gwt/client/ui/VTextField.java
@@ -1,14 +1,17 @@
-/* 
+/*
 @ITMillApache2LicenseForJavaFiles@
  */
 
 package com.vaadin.terminal.gwt.client.ui;
 
+import com.google.gwt.event.dom.client.BlurEvent;
+import com.google.gwt.event.dom.client.BlurHandler;
+import com.google.gwt.event.dom.client.FocusEvent;
+import com.google.gwt.event.dom.client.FocusHandler;
 import com.google.gwt.user.client.DOM;
 import com.google.gwt.user.client.Element;
 import com.google.gwt.user.client.Event;
 import com.google.gwt.user.client.ui.ChangeListener;
-import com.google.gwt.user.client.ui.FocusListener;
 import com.google.gwt.user.client.ui.TextBoxBase;
 import com.google.gwt.user.client.ui.Widget;
 import com.vaadin.terminal.gwt.client.ApplicationConnection;
@@ -20,12 +23,12 @@ import com.vaadin.terminal.gwt.client.VTooltip;
 
 /**
  * This class represents a basic text input field with one row.
- * 
+ *
  * @author IT Mill Ltd.
- * 
+ *
  */
 public class VTextField extends TextBoxBase implements Paintable, Field,
-        ChangeListener, FocusListener {
+        ChangeListener, FocusHandler, BlurHandler {
 
     /**
      * The input node CSS classname.
@@ -65,7 +68,8 @@ public class VTextField extends TextBoxBase implements Paintable, Field,
         }
         setStyleName(CLASSNAME);
         addChangeListener(this);
-        addFocusListener(this);
+        addFocusHandler(this);
+        addBlurHandler(this);
         sinkEvents(VTooltip.TOOLTIP_EVENTS);
     }
 
@@ -158,7 +162,7 @@ public class VTextField extends TextBoxBase implements Paintable, Field,
         }
     }
 
-    public void onFocus(Widget sender) {
+    public void onFocus(FocusEvent event) {
         addStyleDependentName(CLASSNAME_FOCUS);
         if (prompting) {
             setText("");
@@ -167,7 +171,7 @@ public class VTextField extends TextBoxBase implements Paintable, Field,
         focusedTextField = this;
     }
 
-    public void onLostFocus(Widget sender) {
+    public void onBlur(BlurEvent event) {
         removeStyleDependentName(CLASSNAME_FOCUS);
         focusedTextField = null;
         String text = getText();
@@ -176,7 +180,7 @@ public class VTextField extends TextBoxBase implements Paintable, Field,
             setText(inputPrompt);
             addStyleDependentName(CLASSNAME_PROMPT);
         }
-        onChange(sender);
+        onChange((Widget) event.getSource());
     }
 
     private void setPrompting(boolean prompting) {

diff --git a/src/com/vaadin/terminal/gwt/client/ui/VView.java b/src/com/vaadin/terminal/gwt/client/ui/VView.java
index a76f559c3e8dd6e7383449120d6070b8bc999a19..726ef27c797e149cdd0fee8ca309f7403324ce00 100644 (file)
--- a/src/com/vaadin/terminal/gwt/client/ui/VView.java
+++ b/src/com/vaadin/terminal/gwt/client/ui/VView.java
@@ -1,4 +1,4 @@
-/* 
+/*
 @ITMillApache2LicenseForJavaFiles@
  */
 
@@ -21,7 +21,6 @@ import com.google.gwt.user.client.Timer;
 import com.google.gwt.user.client.Window;
 import com.google.gwt.user.client.WindowCloseListener;
 import com.google.gwt.user.client.WindowResizeListener;
-import com.google.gwt.user.client.ui.HasFocus;
 import com.google.gwt.user.client.ui.RootPanel;
 import com.google.gwt.user.client.ui.SimplePanel;
 import com.google.gwt.user.client.ui.Widget;
@@ -35,7 +34,7 @@ import com.vaadin.terminal.gwt.client.UIDL;
 import com.vaadin.terminal.gwt.client.Util;
 
 /**
- * 
+ *
  */
 public class VView extends SimplePanel implements Container,
         WindowResizeListener, WindowCloseListener {
@@ -128,7 +127,7 @@ public class VView extends SimplePanel implements Container,
         try {
             el.focus();
         } catch (e) {
-        
+
         }
     }-*/;
 
@@ -141,7 +140,7 @@ public class VView extends SimplePanel implements Container,
      */
     private static native void reloadHostPage()
     /*-{
-         $wnd.location.reload(); 
+         $wnd.location.reload();
      }-*/;
 
     public void updateFromUIDL(UIDL uidl, ApplicationConnection client) {
@@ -317,8 +316,8 @@ public class VView extends SimplePanel implements Container,
                      * GWT HasFocus of a thinner Vaadin specific Focusable
                      * interface.
                      */
-                    if (toBeFocused instanceof HasFocus) {
-                        final HasFocus toBeFocusedWidget = (HasFocus) toBeFocused;
+                    if (toBeFocused instanceof com.google.gwt.user.client.ui.Focusable) {
+                        final com.google.gwt.user.client.ui.Focusable toBeFocusedWidget = (com.google.gwt.user.client.ui.Focusable) toBeFocused;
                         toBeFocusedWidget.setFocus(true);
                     } else if (toBeFocused instanceof Focusable) {
                         ((Focusable) toBeFocused).focus();
@@ -603,7 +602,7 @@ public class VView extends SimplePanel implements Container,
     /**
      * Return an iterator for current subwindows. This method is meant for
      * testing purposes only.
-     * 
+     *
      * @return
      */
     public ArrayList<VWindow> getSubWindowList() {

diff --git a/src/com/vaadin/terminal/gwt/client/ui/richtextarea/VRichTextArea.java b/src/com/vaadin/terminal/gwt/client/ui/richtextarea/VRichTextArea.java
index 55c1cbf4aa3f544c89b2d822aaebd5a9a98a9124..8f6f3cc97788ebe534b96716feaf688a6ac03715 100644 (file)
--- a/src/com/vaadin/terminal/gwt/client/ui/richtextarea/VRichTextArea.java
+++ b/src/com/vaadin/terminal/gwt/client/ui/richtextarea/VRichTextArea.java
@@ -1,9 +1,11 @@
-/* 
+/*
 @ITMillApache2LicenseForJavaFiles@
  */
 
 package com.vaadin.terminal.gwt.client.ui.richtextarea;
 
+import com.google.gwt.event.dom.client.BlurEvent;
+import com.google.gwt.event.dom.client.BlurHandler;
 import com.google.gwt.user.client.Command;
 import com.google.gwt.user.client.DOM;
 import com.google.gwt.user.client.DeferredCommand;
@@ -11,7 +13,6 @@ import com.google.gwt.user.client.Element;
 import com.google.gwt.user.client.ui.ChangeListener;
 import com.google.gwt.user.client.ui.Composite;
 import com.google.gwt.user.client.ui.FlowPanel;
-import com.google.gwt.user.client.ui.FocusListener;
 import com.google.gwt.user.client.ui.HTML;
 import com.google.gwt.user.client.ui.KeyboardListener;
 import com.google.gwt.user.client.ui.RichTextArea;
@@ -25,12 +26,12 @@ import com.vaadin.terminal.gwt.client.ui.Field;
 
 /**
  * This class implements a basic client side rich text editor component.
- * 
+ *
  * @author IT Mill Ltd.
- * 
+ *
  */
 public class VRichTextArea extends Composite implements Paintable, Field,
-        ChangeListener, FocusListener, KeyboardListener {
+        ChangeListener, BlurHandler, KeyboardListener {
 
     /**
      * The input node CSS classname.
@@ -64,7 +65,7 @@ public class VRichTextArea extends Composite implements Paintable, Field,
         fp.add(formatter);
 
         rta.setWidth("100%");
-        rta.addFocusListener(this);
+        rta.addBlurHandler(this);
 
         fp.add(rta);
 
@@ -131,11 +132,7 @@ public class VRichTextArea extends Composite implements Paintable, Field,
         }
     }
 
-    public void onFocus(Widget sender) {
-
-    }
-
-    public void onLostFocus(Widget sender) {
+    public void onBlur(BlurEvent event) {
         final String html = rta.getHTML();
         client.updateVariable(id, "text", html, immediate);