#3012 Remove deprecated GWT listener use, first steps (many simple click and keyboard...

svn changeset:8580/svn branch:6.1

diff --git a/src/com/vaadin/demo/sampler/gwt/client/ui/VActiveLink.java b/src/com/vaadin/demo/sampler/gwt/client/ui/VActiveLink.java
index a15908ff1e240da4c638d7d4595bef579a842bf9..3ba416bad9c2ac4d194878ed566ebbba07c5c40f 100644 (file)
--- a/src/com/vaadin/demo/sampler/gwt/client/ui/VActiveLink.java
+++ b/src/com/vaadin/demo/sampler/gwt/client/ui/VActiveLink.java
@@ -1,5 +1,6 @@
 package com.vaadin.demo.sampler.gwt.client.ui;
 
+import com.google.gwt.event.dom.client.ClickEvent;
 import com.google.gwt.user.client.DOM;
 import com.google.gwt.user.client.Event;
 import com.google.gwt.user.client.ui.MouseListener;
@@ -41,7 +42,7 @@ public class VActiveLink extends VLink {
     /**
      * Sends variables, returns true if default handler should be called (i.e if
      * server is listening and the link was claimed to be opened by the client)
-     * 
+     *
      * @return
      */
     private boolean sendVariables() {
@@ -63,11 +64,11 @@ public class VActiveLink extends VLink {
     }
 
     @Override
-    public void onClick(Widget sender) {
+    public void onClick(ClickEvent event) {
 
         if (sendVariables()) {
             // run default if not listening, or we claimed link was opened
-            super.onClick(sender);
+            super.onClick(event);
         }
     }
 

diff --git a/src/com/vaadin/terminal/gwt/client/VDebugConsole.java b/src/com/vaadin/terminal/gwt/client/VDebugConsole.java
index fcc0cecf04a87438e663318b58d27f6102c7306a..fb5a81a86cd283952d4654ec5aabce6a320b5ac7 100755 (executable)
--- a/src/com/vaadin/terminal/gwt/client/VDebugConsole.java
+++ b/src/com/vaadin/terminal/gwt/client/VDebugConsole.java
@@ -1,4 +1,4 @@
-/* 
+/*
 @ITMillApache2LicenseForJavaFiles@
  */
 
@@ -8,6 +8,8 @@ import java.util.List;
 import java.util.Set;
 
 import com.google.gwt.core.client.JsArray;
+import com.google.gwt.event.dom.client.ClickEvent;
+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;
@@ -16,7 +18,6 @@ import com.google.gwt.user.client.Window;
 import com.google.gwt.user.client.Window.Location;
 import com.google.gwt.user.client.ui.Button;
 import com.google.gwt.user.client.ui.CheckBox;
-import com.google.gwt.user.client.ui.ClickListener;
 import com.google.gwt.user.client.ui.FlowPanel;
 import com.google.gwt.user.client.ui.HTML;
 import com.google.gwt.user.client.ui.HorizontalPanel;
@@ -109,8 +110,8 @@ public final class VDebugConsole extends VOverlay implements Console {
 
             panel.add(new HTML("<i>" + help + "</i>"));
 
-            clear.addClickListener(new ClickListener() {
-                public void onClick(Widget sender) {
+            clear.addClickHandler(new ClickHandler() {
+                public void onClick(ClickEvent event) {
                     int width = panel.getOffsetWidth();
                     int height = panel.getOffsetHeight();
                     panel = new FlowPanel();
@@ -121,8 +122,8 @@ public final class VDebugConsole extends VOverlay implements Console {
                 }
             });
 
-            restart.addClickListener(new ClickListener() {
-                public void onClick(Widget sender) {
+            restart.addClickHandler(new ClickHandler() {
+                public void onClick(ClickEvent event) {
 
                     String queryString = Window.Location.getQueryString();
                     if (queryString != null
@@ -148,14 +149,14 @@ public final class VDebugConsole extends VOverlay implements Console {
                 }
             });
 
-            forceLayout.addClickListener(new ClickListener() {
-                public void onClick(Widget sender) {
+            forceLayout.addClickHandler(new ClickHandler() {
+                public void onClick(ClickEvent event) {
                     VDebugConsole.this.client.forceLayout();
                 }
             });
 
-            analyzeLayout.addClickListener(new ClickListener() {
-                public void onClick(Widget sender) {
+            analyzeLayout.addClickHandler(new ClickHandler() {
+                public void onClick(ClickEvent event) {
                     List<ApplicationConnection> runningApplications = ApplicationConfiguration
                             .getRunningApplications();
                     for (ApplicationConnection applicationConnection : runningApplications) {
@@ -291,7 +292,7 @@ public final class VDebugConsole extends VOverlay implements Console {
 
     /*
      * (non-Javadoc)
-     * 
+     *
      * @see com.vaadin.terminal.gwt.client.Console#log(java.lang.String)
      */
     public void log(String msg) {
@@ -302,7 +303,7 @@ public final class VDebugConsole extends VOverlay implements Console {
 
     /*
      * (non-Javadoc)
-     * 
+     *
      * @see com.vaadin.terminal.gwt.client.Console#error(java.lang.String)
      */
     public void error(String msg) {
@@ -313,7 +314,7 @@ public final class VDebugConsole extends VOverlay implements Console {
 
     /*
      * (non-Javadoc)
-     * 
+     *
      * @see com.vaadin.terminal.gwt.client.Console#printObject(java.lang.
      * Object)
      */
@@ -324,7 +325,7 @@ public final class VDebugConsole extends VOverlay implements Console {
 
     /*
      * (non-Javadoc)
-     * 
+     *
      * @see com.vaadin.terminal.gwt.client.Console#dirUIDL(com.vaadin
      * .terminal.gwt.client.UIDL)
      */
@@ -345,7 +346,7 @@ public final class VDebugConsole extends VOverlay implements Console {
                  $wnd.console.log(u);
              }
          }
-        
+
     }-*/;
 
     private static native void consoleLog(String msg)
@@ -418,12 +419,12 @@ public final class VDebugConsole extends VOverlay implements Console {
                     + " inside " + Util.getSimpleName(layout)));
             final CheckBox emphasisInUi = new CheckBox(
                     "Emphasis components parent in UI (actual component is not visible)");
-            emphasisInUi.addClickListener(new ClickListener() {
-                public void onClick(Widget sender) {
+            emphasisInUi.addClickHandler(new ClickHandler() {
+                public void onClick(ClickEvent event) {
                     if (paintable != null) {
                         Element element2 = ((Widget) layout).getElement();
                         Widget.setStyleName(element2, "invalidlayout",
-                                emphasisInUi.isChecked());
+                                emphasisInUi.getValue());
                     }
                 }
             });
@@ -450,12 +451,12 @@ public final class VDebugConsole extends VOverlay implements Console {
                     + valueMap.getString("widthMsg")));
         }
         final CheckBox emphasisInUi = new CheckBox("Emphasis component in UI");
-        emphasisInUi.addClickListener(new ClickListener() {
-            public void onClick(Widget sender) {
+        emphasisInUi.addClickHandler(new ClickHandler() {
+            public void onClick(ClickEvent event) {
                 if (paintable != null) {
                     Element element2 = ((Widget) paintable).getElement();
                     Widget.setStyleName(element2, "invalidlayout", emphasisInUi
-                            .isChecked());
+                            .getValue());
                 }
             }
         });

diff --git a/src/com/vaadin/terminal/gwt/client/ui/VAccordion.java b/src/com/vaadin/terminal/gwt/client/ui/VAccordion.java
index 6e9d4512be7b0c50eaa1d8b0f79e56fe9799f960..a7fa36769f365beade3a2b7009ffadb7dbb40c79 100644 (file)
--- a/src/com/vaadin/terminal/gwt/client/ui/VAccordion.java
+++ b/src/com/vaadin/terminal/gwt/client/ui/VAccordion.java
@@ -5,10 +5,11 @@ import java.util.HashSet;
 import java.util.Iterator;
 import java.util.Set;
 
+import com.google.gwt.event.dom.client.ClickEvent;
+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.ClickListener;
 import com.google.gwt.user.client.ui.ComplexPanel;
 import com.google.gwt.user.client.ui.Widget;
 import com.vaadin.terminal.gwt.client.ApplicationConnection;
@@ -124,7 +125,7 @@ public class VAccordion extends VTabsheetBase implements
      * same StackItem is used for rendering this time. E.g. if the first tab has
      * been removed all tabs which contain cached content must be moved 1 step
      * up to preserve the cached content.
-     * 
+     *
      * @param item
      * @param newIndex
      * @param tabUidl
@@ -352,9 +353,9 @@ public class VAccordion extends VTabsheetBase implements
     }
 
     /**
-     * 
+     *
      */
-    protected class StackItem extends ComplexPanel implements ClickListener {
+    protected class StackItem extends ComplexPanel implements ClickHandler {
 
         public void setHeight(int height) {
             if (height == -1) {
@@ -395,7 +396,7 @@ public class VAccordion extends VTabsheetBase implements
 
         /**
          * Returns caption width including padding
-         * 
+         *
          * @return
          */
         public int getCaptionWidth() {
@@ -433,7 +434,7 @@ public class VAccordion extends VTabsheetBase implements
         public StackItem(UIDL tabUidl) {
             setElement(DOM.createDiv());
             caption = new VCaption(null, client);
-            caption.addClickListener(this);
+            caption.addClickHandler(this);
             if (BrowserInfo.get().isIE6()) {
                 DOM.setEventListener(captionNode, this);
                 DOM.sinkEvents(captionNode, Event.BUTTON_LEFT);
@@ -523,7 +524,7 @@ public class VAccordion extends VTabsheetBase implements
             }
         }
 
-        public void onClick(Widget sender) {
+        public void onClick(ClickEvent event) {
             onSelectTab(this);
         }
 

diff --git a/src/com/vaadin/terminal/gwt/client/ui/VCheckBox.java b/src/com/vaadin/terminal/gwt/client/ui/VCheckBox.java
index 42978a26391ee4af579490c01b3bcd12210478c3..ec460d1e5e9964990af72efa05832261be5ac7a5 100644 (file)
--- a/src/com/vaadin/terminal/gwt/client/ui/VCheckBox.java
+++ b/src/com/vaadin/terminal/gwt/client/ui/VCheckBox.java
@@ -1,14 +1,14 @@
-/* 
+/*
 @ITMillApache2LicenseForJavaFiles@
  */
 
 package com.vaadin.terminal.gwt.client.ui;
 
+import com.google.gwt.event.dom.client.ClickEvent;
+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.ClickListener;
-import com.google.gwt.user.client.ui.Widget;
 import com.vaadin.terminal.gwt.client.ApplicationConnection;
 import com.vaadin.terminal.gwt.client.Paintable;
 import com.vaadin.terminal.gwt.client.UIDL;
@@ -34,13 +34,13 @@ public class VCheckBox extends com.google.gwt.user.client.ui.CheckBox implements
 
     public VCheckBox() {
         setStyleName(CLASSNAME);
-        addClickListener(new ClickListener() {
+        addClickHandler(new ClickHandler() {
 
-            public void onClick(Widget sender) {
+            public void onClick(ClickEvent event) {
                 if (id == null || client == null) {
                     return;
                 }
-                client.updateVariable(id, "state", isChecked(), immediate);
+                client.updateVariable(id, "state", getValue(), immediate);
             }
 
         });
@@ -97,15 +97,15 @@ public class VCheckBox extends com.google.gwt.user.client.ui.CheckBox implements
 
         // Set text
         setText(uidl.getStringAttribute("caption"));
-        setChecked(uidl.getBooleanVariable("state"));
+        setValue(uidl.getBooleanVariable("state"));
         immediate = uidl.getBooleanAttribute("immediate");
     }
 
     @Override
     public void onBrowserEvent(Event event) {
         if (icon != null && (event.getTypeInt() == Event.ONCLICK)
-                && (event.getTarget() == icon.getElement())) {
-            setChecked(!isChecked());
+                && (DOM.eventGetTarget(event) == icon.getElement())) {
+            setValue(!getValue());
         }
         super.onBrowserEvent(event);
         if (event.getTypeInt() == Event.ONLOAD) {

diff --git a/src/com/vaadin/terminal/gwt/client/ui/VFilterSelect.java b/src/com/vaadin/terminal/gwt/client/ui/VFilterSelect.java
index e57d009724ea2dfd2134bc466876a2380e2e1a33..bdea4fbbe792a7f5de929551444c63a35dea6e17 100644 (file)
--- a/src/com/vaadin/terminal/gwt/client/ui/VFilterSelect.java
+++ b/src/com/vaadin/terminal/gwt/client/ui/VFilterSelect.java
@@ -1,4 +1,4 @@
-/* 
+/*
 @ITMillApache2LicenseForJavaFiles@
  */
 
@@ -10,18 +10,23 @@ import java.util.Date;
 import java.util.Iterator;
 import java.util.List;
 
+import com.google.gwt.event.dom.client.ClickEvent;
+import com.google.gwt.event.dom.client.ClickHandler;
+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.user.client.Command;
 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.Window;
-import com.google.gwt.user.client.ui.ClickListener;
 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.KeyboardListener;
 import com.google.gwt.user.client.ui.LoadListener;
 import com.google.gwt.user.client.ui.PopupListener;
 import com.google.gwt.user.client.ui.PopupPanel;
@@ -38,11 +43,11 @@ import com.vaadin.terminal.gwt.client.Util;
 import com.vaadin.terminal.gwt.client.VTooltip;
 
 /**
- * 
+ *
  * TODO needs major refactoring (to be extensible etc)
  */
 public class VFilterSelect extends Composite implements Paintable, Field,
-        KeyboardListener, ClickListener, FocusListener, Focusable {
+        KeyDownHandler, KeyUpHandler, ClickHandler, FocusListener, Focusable {
 
     public class FilterSelectSuggestion implements Suggestion, Command {
 
@@ -276,7 +281,7 @@ public class VFilterSelect extends Composite implements Paintable, Field,
 
         /*
          * (non-Javadoc)
-         * 
+         *
          * @see
          * com.google.gwt.user.client.ui.PopupPanel$PositionCallback#setPosition
          * (int, int)
@@ -599,11 +604,12 @@ public class VFilterSelect extends Composite implements Paintable, Field,
         panel.add(popupOpener);
         initWidget(panel);
         setStyleName(CLASSNAME);
-        tb.addKeyboardListener(this);
+        tb.addKeyDownHandler(this);
+        tb.addKeyUpHandler(this);
         tb.setStyleName(CLASSNAME + "-input");
         tb.addFocusListener(this);
         popupOpener.setStyleName(CLASSNAME + "-button");
-        popupOpener.addClickListener(this);
+        popupOpener.addClickHandler(this);
     }
 
     public boolean hasNextPage() {
@@ -840,22 +846,22 @@ public class VFilterSelect extends Composite implements Paintable, Field,
                 marginTop + "px");
     }
 
-    public void onKeyDown(Widget sender, char keyCode, int modifiers) {
+    public void onKeyDown(KeyDownEvent event) {
         if (enabled) {
             if (suggestionPopup.isAttached()) {
-                popupKeyDown(keyCode);
+                popupKeyDown(event);
             } else {
-                inputFieldKeyDown(keyCode);
+                inputFieldKeyDown(event);
             }
         }
     }
 
-    private void inputFieldKeyDown(char keyCode) {
-        switch (keyCode) {
-        case KeyboardListener.KEY_DOWN:
-        case KeyboardListener.KEY_UP:
-        case KeyboardListener.KEY_PAGEDOWN:
-        case KeyboardListener.KEY_PAGEUP:
+    private void inputFieldKeyDown(KeyDownEvent event) {
+        switch (event.getNativeKeyCode()) {
+        case KeyCodes.KEY_DOWN:
+        case KeyCodes.KEY_UP:
+        case KeyCodes.KEY_PAGEDOWN:
+        case KeyCodes.KEY_PAGEUP:
             if (suggestionPopup.isAttached()) {
                 break;
             } else {
@@ -869,53 +875,49 @@ public class VFilterSelect extends Composite implements Paintable, Field,
 
     }
 
-    private void popupKeyDown(char keyCode) {
-        switch (keyCode) {
-        case KeyboardListener.KEY_DOWN:
+    private void popupKeyDown(KeyDownEvent event) {
+        switch (event.getNativeKeyCode()) {
+        case KeyCodes.KEY_DOWN:
             suggestionPopup.selectNextItem();
             DOM.eventPreventDefault(DOM.eventGetCurrentEvent());
             break;
-        case KeyboardListener.KEY_UP:
+        case KeyCodes.KEY_UP:
             suggestionPopup.selectPrevItem();
             DOM.eventPreventDefault(DOM.eventGetCurrentEvent());
             break;
-        case KeyboardListener.KEY_PAGEDOWN:
+        case KeyCodes.KEY_PAGEDOWN:
             if (hasNextPage()) {
                 filterOptions(currentPage + 1, lastFilter);
             }
             break;
-        case KeyboardListener.KEY_PAGEUP:
+        case KeyCodes.KEY_PAGEUP:
             if (currentPage > 0) {
                 filterOptions(currentPage - 1, lastFilter);
             }
             break;
-        case KeyboardListener.KEY_ENTER:
-        case KeyboardListener.KEY_TAB:
+        case KeyCodes.KEY_ENTER:
+        case KeyCodes.KEY_TAB:
             suggestionPopup.menu.doSelectedItemAction();
             break;
         }
 
     }
 
-    public void onKeyPress(Widget sender, char keyCode, int modifiers) {
-
-    }
-
-    public void onKeyUp(Widget sender, char keyCode, int modifiers) {
+    public void onKeyUp(KeyUpEvent event) {
         if (enabled) {
-            switch (keyCode) {
-            case KeyboardListener.KEY_ENTER:
-            case KeyboardListener.KEY_TAB:
-            case KeyboardListener.KEY_SHIFT:
-            case KeyboardListener.KEY_CTRL:
-            case KeyboardListener.KEY_ALT:
-            case KeyboardListener.KEY_DOWN:
-            case KeyboardListener.KEY_UP:
-            case KeyboardListener.KEY_PAGEDOWN:
-            case KeyboardListener.KEY_PAGEUP:
+            switch (event.getNativeKeyCode()) {
+            case KeyCodes.KEY_ENTER:
+            case KeyCodes.KEY_TAB:
+            case KeyCodes.KEY_SHIFT:
+            case KeyCodes.KEY_CTRL:
+            case KeyCodes.KEY_ALT:
+            case KeyCodes.KEY_DOWN:
+            case KeyCodes.KEY_UP:
+            case KeyCodes.KEY_PAGEDOWN:
+            case KeyCodes.KEY_PAGEUP:
                 ; // NOP
                 break;
-            case KeyboardListener.KEY_ESCAPE:
+            case KeyCodes.KEY_ESCAPE:
                 reset();
                 break;
             default:
@@ -941,7 +943,7 @@ public class VFilterSelect extends Composite implements Paintable, Field,
     /**
      * Listener for popupopener
      */
-    public void onClick(Widget sender) {
+    public void onClick(ClickEvent event) {
         if (enabled) {
             // ask suggestionPopup if it was just closed, we are using GWT
             // Popup's auto close feature

diff --git a/src/com/vaadin/terminal/gwt/client/ui/VLink.java b/src/com/vaadin/terminal/gwt/client/ui/VLink.java
index 790f8cbf5ef84a19d8b6a7ebd28387316e61ba86..22a30f604152f4101269ee6e55d515bf8de6e314 100644 (file)
--- a/src/com/vaadin/terminal/gwt/client/ui/VLink.java
+++ b/src/com/vaadin/terminal/gwt/client/ui/VLink.java
@@ -1,23 +1,23 @@
-/* 
+/*
 @ITMillApache2LicenseForJavaFiles@
  */
 
 package com.vaadin.terminal.gwt.client.ui;
 
+import com.google.gwt.event.dom.client.ClickEvent;
+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.Window;
-import com.google.gwt.user.client.ui.ClickListener;
 import com.google.gwt.user.client.ui.HTML;
-import com.google.gwt.user.client.ui.Widget;
 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 VLink extends HTML implements Paintable, ClickListener {
+public class VLink extends HTML implements Paintable, ClickHandler {
 
     public static final String CLASSNAME = "v-link";
 
@@ -53,7 +53,7 @@ public class VLink extends HTML implements Paintable, ClickListener {
         super();
         getElement().appendChild(anchor);
         anchor.appendChild(captionElement);
-        addClickListener(this);
+        addClickHandler(this);
         sinkEvents(VTooltip.TOOLTIP_EVENTS);
         setStyleName(CLASSNAME);
     }
@@ -120,7 +120,7 @@ public class VLink extends HTML implements Paintable, ClickListener {
 
     }
 
-    public void onClick(Widget sender) {
+    public void onClick(ClickEvent event) {
         if (enabled && !readonly) {
             if (target == null) {
                 target = "_self";

diff --git a/src/com/vaadin/terminal/gwt/client/ui/VListSelect.java b/src/com/vaadin/terminal/gwt/client/ui/VListSelect.java
index 17c342dcdcfb6122b51c8416ba31cf89739ebd67..0e519c446005d09dd92df603f342237e3adf3c2f 100644 (file)
--- a/src/com/vaadin/terminal/gwt/client/ui/VListSelect.java
+++ b/src/com/vaadin/terminal/gwt/client/ui/VListSelect.java
@@ -1,4 +1,4 @@
-/* 
+/*
 @ITMillApache2LicenseForJavaFiles@
  */
 
@@ -30,7 +30,7 @@ public class VListSelect extends VOptionGroupBase {
         select = (TooltipListBox) optionsContainer;
         select.setSelect(this);
         select.addChangeListener(this);
-        select.addClickListener(this);
+        select.addClickHandler(this);
         select.setStyleName(CLASSNAME + "-select");
         select.setVisibleItemCount(VISIBLE_COUNT);
     }

diff --git a/src/com/vaadin/terminal/gwt/client/ui/VOptionGroup.java b/src/com/vaadin/terminal/gwt/client/ui/VOptionGroup.java
index 4940af65ab763beb4d9753e287f4f959a6c7dc24..2f76ee1bef68a5343321c06b6bba9d692aff0303 100644 (file)
--- a/src/com/vaadin/terminal/gwt/client/ui/VOptionGroup.java
+++ b/src/com/vaadin/terminal/gwt/client/ui/VOptionGroup.java
@@ -1,4 +1,4 @@
-/* \r
+/*\r
 @ITMillApache2LicenseForJavaFiles@\r
  */\r
 \r
@@ -8,6 +8,7 @@ import java.util.HashMap;
 import java.util.Iterator;\r
 import java.util.Map;\r
 \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.Panel;\r
@@ -49,7 +50,7 @@ public class VOptionGroup extends VOptionGroupBase {
             op.setChecked(opUidl.getBooleanAttribute("selected"));\r
             op.setEnabled(!opUidl.getBooleanAttribute("disabled")\r
                     && !isReadonly() && !isDisabled());\r
-            op.addClickListener(this);\r
+            op.addClickHandler(this);\r
             optionsToKeys.put(op, opUidl.getStringAttribute("key"));\r
             panel.add(op);\r
         }\r
@@ -61,11 +62,11 @@ public class VOptionGroup extends VOptionGroupBase {
     }\r
 \r
     @Override\r
-    public void onClick(Widget sender) {\r
-        super.onClick(sender);\r
-        if (sender instanceof CheckBox) {\r
-            final boolean selected = ((CheckBox) sender).isChecked();\r
-            final String key = (String) optionsToKeys.get(sender);\r
+    public void onClick(ClickEvent event) {\r
+        super.onClick(event);\r
+        if (event.getSource() instanceof CheckBox) {\r
+            final boolean selected = ((CheckBox) event.getSource()).getValue();\r
+            final String key = (String) optionsToKeys.get(event.getSource());\r
             if (!isMultiselect()) {\r
                 selectedKeys.clear();\r
             }\r

diff --git a/src/com/vaadin/terminal/gwt/client/ui/VOptionGroupBase.java b/src/com/vaadin/terminal/gwt/client/ui/VOptionGroupBase.java
index 4d4ba4e57929ee4459dd62cce55ca6a5fd511a65..a4bb9e9b4e711bf21ca8a135e6018d1dd4729550 100644 (file)
--- a/src/com/vaadin/terminal/gwt/client/ui/VOptionGroupBase.java
+++ b/src/com/vaadin/terminal/gwt/client/ui/VOptionGroupBase.java
@@ -1,4 +1,4 @@
-/* \r
+/*\r
 @ITMillApache2LicenseForJavaFiles@\r
  */\r
 \r
@@ -6,11 +6,14 @@ package com.vaadin.terminal.gwt.client.ui;
 \r
 import java.util.Set;\r
 \r
+import com.google.gwt.event.dom.client.ClickEvent;\r
+import com.google.gwt.event.dom.client.ClickHandler;\r
+import com.google.gwt.event.dom.client.KeyCodes;\r
+import com.google.gwt.event.dom.client.KeyPressEvent;\r
+import com.google.gwt.event.dom.client.KeyPressHandler;\r
 import com.google.gwt.user.client.ui.ChangeListener;\r
-import com.google.gwt.user.client.ui.ClickListener;\r
 import com.google.gwt.user.client.ui.Composite;\r
 import com.google.gwt.user.client.ui.FlowPanel;\r
-import com.google.gwt.user.client.ui.KeyboardListener;\r
 import com.google.gwt.user.client.ui.Panel;\r
 import com.google.gwt.user.client.ui.Widget;\r
 import com.vaadin.terminal.gwt.client.ApplicationConnection;\r
@@ -19,7 +22,7 @@ import com.vaadin.terminal.gwt.client.Paintable;
 import com.vaadin.terminal.gwt.client.UIDL;\r
 \r
 abstract class VOptionGroupBase extends Composite implements Paintable, Field,\r
-        ClickListener, ChangeListener, KeyboardListener, Focusable {\r
+        ClickHandler, ChangeListener, KeyPressHandler, Focusable {\r
 \r
     public static final String CLASSNAME_OPTION = "v-select-option";\r
 \r
@@ -157,9 +160,9 @@ abstract class VOptionGroupBase extends Composite implements Paintable, Field,
             if (newItemField == null) {\r
                 newItemButton = new VNativeButton();\r
                 newItemButton.setText("+");\r
-                newItemButton.addClickListener(this);\r
+                newItemButton.addClickHandler(this);\r
                 newItemField = new VTextField();\r
-                newItemField.addKeyboardListener(this);\r
+                newItemField.addKeyPressHandler(this);\r
             }\r
             newItemField.setEnabled(!disabled && !readonly);\r
             newItemButton.setEnabled(!disabled && !readonly);\r
@@ -183,8 +186,9 @@ abstract class VOptionGroupBase extends Composite implements Paintable, Field,
 \r
     abstract protected void setTabIndex(int tabIndex);\r
 \r
-    public void onClick(Widget sender) {\r
-        if (sender == newItemButton && !newItemField.getText().equals("")) {\r
+    public void onClick(ClickEvent event) {\r
+        if (event.getSource() == newItemButton\r
+                && !newItemField.getText().equals("")) {\r
             client.updateVariable(id, "newitem", newItemField.getText(), true);\r
             newItemField.setText("");\r
         }\r
@@ -201,20 +205,13 @@ abstract class VOptionGroupBase extends Composite implements Paintable, Field,
         }\r
     }\r
 \r
-    public void onKeyPress(Widget sender, char keyCode, int modifiers) {\r
-        if (sender == newItemField && keyCode == KeyboardListener.KEY_ENTER) {\r
+    public void onKeyPress(KeyPressEvent event) {\r
+        if (event.getSource() == newItemField\r
+                && event.getCharCode() == KeyCodes.KEY_ENTER) {\r
             newItemButton.click();\r
         }\r
     }\r
 \r
-    public void onKeyUp(Widget sender, char keyCode, int modifiers) {\r
-        // Ignore, subclasses may override\r
-    }\r
-\r
-    public void onKeyDown(Widget sender, char keyCode, int modifiers) {\r
-        // Ignore, subclasses may override\r
-    }\r
-\r
     protected abstract void buildOptions(UIDL uidl);\r
 \r
     protected abstract Object[] getSelectedItems();\r

diff --git a/src/com/vaadin/terminal/gwt/client/ui/VPopupCalendar.java b/src/com/vaadin/terminal/gwt/client/ui/VPopupCalendar.java
index ea18b5824a749f36e75f7bcd5f1f790ea20ff4ed..feade2e962f364e79334026a3d528b5b75a7b5e4 100644 (file)
--- a/src/com/vaadin/terminal/gwt/client/ui/VPopupCalendar.java
+++ b/src/com/vaadin/terminal/gwt/client/ui/VPopupCalendar.java
@@ -1,24 +1,24 @@
-/* \r
+/*\r
 @ITMillApache2LicenseForJavaFiles@\r
  */\r
 \r
 package com.vaadin.terminal.gwt.client.ui;\r
 \r
+import com.google.gwt.event.dom.client.ClickEvent;\r
+import com.google.gwt.event.dom.client.ClickHandler;\r
 import com.google.gwt.user.client.DOM;\r
 import com.google.gwt.user.client.Timer;\r
 import com.google.gwt.user.client.Window;\r
 import com.google.gwt.user.client.ui.Button;\r
-import com.google.gwt.user.client.ui.ClickListener;\r
 import com.google.gwt.user.client.ui.PopupListener;\r
 import com.google.gwt.user.client.ui.PopupPanel;\r
-import com.google.gwt.user.client.ui.Widget;\r
 import com.google.gwt.user.client.ui.PopupPanel.PositionCallback;\r
 import com.vaadin.terminal.gwt.client.ApplicationConnection;\r
 import com.vaadin.terminal.gwt.client.Paintable;\r
 import com.vaadin.terminal.gwt.client.UIDL;\r
 \r
 public class VPopupCalendar extends VTextualDate implements Paintable, Field,\r
-        ClickListener, PopupListener {\r
+        ClickHandler, PopupListener {\r
 \r
     private final Button calendarToggle;\r
 \r
@@ -33,7 +33,7 @@ public class VPopupCalendar extends VTextualDate implements Paintable, Field,
         calendarToggle = new Button();\r
         calendarToggle.setStyleName(CLASSNAME + "-button");\r
         calendarToggle.setText("");\r
-        calendarToggle.addClickListener(this);\r
+        calendarToggle.addClickHandler(this);\r
         add(calendarToggle);\r
 \r
         calendar = new VCalendarPanel(this);\r
@@ -60,8 +60,8 @@ public class VPopupCalendar extends VTextualDate implements Paintable, Field,
         calendarToggle.setEnabled(enabled);\r
     }\r
 \r
-    public void onClick(Widget sender) {\r
-        if (sender == calendarToggle && !open) {\r
+    public void onClick(ClickEvent event) {\r
+        if (event.getSource() == calendarToggle && !open) {\r
             open = true;\r
             calendar.updateCalendar();\r
             // clear previous values\r
@@ -115,7 +115,7 @@ public class VPopupCalendar extends VTextualDate implements Paintable, Field,
 \r
     /**\r
      * Sets focus to Calendar panel.\r
-     * \r
+     *\r
      * @param focus\r
      */\r
     public void setFocus(boolean focus) {\r

diff --git a/src/com/vaadin/terminal/gwt/client/ui/VPopupView.java b/src/com/vaadin/terminal/gwt/client/ui/VPopupView.java
index 1bdefe63729c65fcdab97a203cbfebde361e5b3d..23fe10f3717220f12537c860edcf583ad8b48aa2 100644 (file)
--- a/src/com/vaadin/terminal/gwt/client/ui/VPopupView.java
+++ b/src/com/vaadin/terminal/gwt/client/ui/VPopupView.java
@@ -3,10 +3,11 @@ package com.vaadin.terminal.gwt.client.ui;
 import java.util.HashSet;
 import java.util.Set;
 
+import com.google.gwt.event.dom.client.ClickEvent;
+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.ClickListener;
 import com.google.gwt.user.client.ui.HTML;
 import com.google.gwt.user.client.ui.HasFocus;
 import com.google.gwt.user.client.ui.Label;
@@ -53,8 +54,8 @@ public class VPopupView extends HTML implements Container {
         popup.setWidget(loading);
 
         // When we click to open the popup...
-        addClickListener(new ClickListener() {
-            public void onClick(Widget sender) {
+        addClickHandler(new ClickHandler() {
+            public void onClick(ClickEvent event) {
                 updateState(true);
             }
         });
@@ -71,8 +72,8 @@ public class VPopupView extends HTML implements Container {
     }
 
     /**
-     * 
-     * 
+     *
+     *
      * @see com.vaadin.terminal.gwt.client.Paintable#updateFromUIDL(com.vaadin.terminal.gwt.client.UIDL,
      *      com.vaadin.terminal.gwt.client.ApplicationConnection)
      */
@@ -127,7 +128,7 @@ public class VPopupView extends HTML implements Container {
 
     /**
      * Update popup visibility to server
-     * 
+     *
      * @param visibility
      */
     private void updateState(boolean visible) {
@@ -184,7 +185,7 @@ public class VPopupView extends HTML implements Container {
 
     /**
      * Make sure that we remove the popup when the main widget is removed.
-     * 
+     *
      * @see com.google.gwt.user.client.ui.Widget#onUnload()
      */
     @Override
@@ -194,7 +195,7 @@ public class VPopupView extends HTML implements Container {
     }
 
     private static native void nativeBlur(Element e)
-    /*-{ 
+    /*-{
         if(e && e.blur) {
             e.blur();
         }
@@ -321,13 +322,13 @@ public class VPopupView extends HTML implements Container {
         }
 
         /*
-         * 
+         *
          * We need a hack make popup act as a child of VPopupView in Vaadin's
          * component tree, but work in default GWT manner when closing or
          * opening.
-         * 
+         *
          * (non-Javadoc)
-         * 
+         *
          * @see com.google.gwt.user.client.ui.Widget#getParent()
          */
         @Override
@@ -364,7 +365,7 @@ public class VPopupView extends HTML implements Container {
 
     /**
      * Calculate extra space taken by the popup decorations
-     * 
+     *
      * @return
      */
     protected Size calculatePopupExtra() {

diff --git a/src/com/vaadin/terminal/gwt/client/ui/VTablePaging.java b/src/com/vaadin/terminal/gwt/client/ui/VTablePaging.java
index 411abff111099441fc24832faf28f9cfae82bbfa..6995513524cc8103b233fd6b9eb009402db959eb 100644 (file)
--- a/src/com/vaadin/terminal/gwt/client/ui/VTablePaging.java
+++ b/src/com/vaadin/terminal/gwt/client/ui/VTablePaging.java
@@ -1,4 +1,4 @@
-/* 
+/*
 @ITMillApache2LicenseForJavaFiles@
  */
 
@@ -9,11 +9,12 @@ import java.util.HashMap;
 import java.util.Iterator;
 import java.util.Set;
 
+import com.google.gwt.event.dom.client.ClickEvent;
+import com.google.gwt.event.dom.client.ClickHandler;
 import com.google.gwt.user.client.DOM;
 import com.google.gwt.user.client.Event;
 import com.google.gwt.user.client.Window;
 import com.google.gwt.user.client.ui.Button;
-import com.google.gwt.user.client.ui.ClickListener;
 import com.google.gwt.user.client.ui.Composite;
 import com.google.gwt.user.client.ui.Grid;
 import com.google.gwt.user.client.ui.HTML;
@@ -31,7 +32,7 @@ import com.vaadin.terminal.gwt.client.UIDL;
  * style table which will be much lighter than VScrollTable is.
  */
 public class VTablePaging extends Composite implements Table, Paintable,
-        ClickListener {
+        ClickHandler {
 
     private final Grid tBody = new Grid();
     private final Button nextPage = new Button(">");
@@ -72,13 +73,13 @@ public class VTablePaging extends Composite implements Table, Paintable,
 
         pager = new HorizontalPanel();
         pager.add(firstPage);
-        firstPage.addClickListener(this);
+        firstPage.addClickHandler(this);
         pager.add(prevPage);
-        prevPage.addClickListener(this);
+        prevPage.addClickHandler(this);
         pager.add(nextPage);
-        nextPage.addClickListener(this);
+        nextPage.addClickHandler(this);
         pager.add(lastPage);
-        lastPage.addClickListener(this);
+        lastPage.addClickHandler(this);
 
         panel.add(pager);
         panel.add(tBody);
@@ -168,10 +169,10 @@ public class VTablePaging extends Composite implements Table, Paintable,
     /**
      * Updates row data from uidl. UpdateFromUIDL delegates updating tBody to
      * this method.
-     * 
+     *
      * Updates may be to different part of tBody, depending on update type. It
      * can be initial row data, scroll up, scroll down...
-     * 
+     *
      * @param uidl
      *            which contains row data
      */
@@ -245,7 +246,8 @@ public class VTablePaging extends Composite implements Table, Paintable,
         return false;
     }
 
-    public void onClick(Widget sender) {
+    public void onClick(ClickEvent event) {
+        Object sender = event.getSource();
         if (sender instanceof Button) {
             if (sender == firstPage) {
                 client.updateVariable(id, "firstvisible", 0, true);
@@ -286,7 +288,7 @@ public class VTablePaging extends Composite implements Table, Paintable,
         HeaderCell(String pid, String caption) {
             super();
             cid = pid;
-            addClickListener(VTablePaging.this);
+            addClickHandler(VTablePaging.this);
             setText(caption);
             // TODO remove debug color
             DOM.setStyleAttribute(getElement(), "color", "brown");
@@ -297,7 +299,7 @@ public class VTablePaging extends Composite implements Table, Paintable,
     /**
      * Abstraction of table cell content. In needs to know on which row it is in
      * case of context click.
-     * 
+     *
      * @author mattitahvonen
      */
     public class BodyCell extends SimplePanel {
@@ -353,7 +355,7 @@ public class VTablePaging extends Composite implements Table, Paintable,
         /**
          * This method is used to set row status. Does not change value on
          * server.
-         * 
+         *
          * @param selected
          */
         public void setSelected(boolean sel) {
@@ -377,7 +379,7 @@ public class VTablePaging extends Composite implements Table, Paintable,
         /**
          * Toggles rows select state. Also updates state to server according to
          * tables immediate flag.
-         * 
+         *
          */
         public void toggleSelected() {
             if (selected) {
@@ -394,7 +396,7 @@ public class VTablePaging extends Composite implements Table, Paintable,
 
         /**
          * Shows context menu for this row.
-         * 
+         *
          * @param event
          *            Event which triggered context menu. Correct place for
          *            context menu can be determined with it.

diff --git a/src/com/vaadin/terminal/gwt/client/ui/VTwinColSelect.java b/src/com/vaadin/terminal/gwt/client/ui/VTwinColSelect.java
index 744fe77a001a38bfcff9b7ad54db1778e2243805..1694992074ddc8f3def5b997c2335989657856e9 100644 (file)
--- a/src/com/vaadin/terminal/gwt/client/ui/VTwinColSelect.java
+++ b/src/com/vaadin/terminal/gwt/client/ui/VTwinColSelect.java
@@ -1,4 +1,4 @@
-/* \r
+/*\r
 @ITMillApache2LicenseForJavaFiles@\r
  */\r
 \r
@@ -7,12 +7,12 @@ package com.vaadin.terminal.gwt.client.ui;
 import java.util.ArrayList;\r
 import java.util.Iterator;\r
 \r
+import com.google.gwt.event.dom.client.ClickEvent;\r
 import com.google.gwt.user.client.DOM;\r
 import com.google.gwt.user.client.ui.FlowPanel;\r
 import com.google.gwt.user.client.ui.HTML;\r
 import com.google.gwt.user.client.ui.ListBox;\r
 import com.google.gwt.user.client.ui.Panel;\r
-import com.google.gwt.user.client.ui.Widget;\r
 import com.vaadin.terminal.gwt.client.UIDL;\r
 \r
 public class VTwinColSelect extends VOptionGroupBase {\r
@@ -40,9 +40,9 @@ public class VTwinColSelect extends VOptionGroupBase {
     public VTwinColSelect() {\r
         super(CLASSNAME);\r
         options = new ListBox();\r
-        options.addClickListener(this);\r
+        options.addClickHandler(this);\r
         selections = new ListBox();\r
-        selections.addClickListener(this);\r
+        selections.addClickHandler(this);\r
         options.setVisibleItemCount(VISIBLE_COUNT);\r
         selections.setVisibleItemCount(VISIBLE_COUNT);\r
         options.setStyleName(CLASSNAME + "-options");\r
@@ -51,10 +51,10 @@ public class VTwinColSelect extends VOptionGroupBase {
         buttons.setStyleName(CLASSNAME + "-buttons");\r
         add = new VButton();\r
         add.setText(">>");\r
-        add.addClickListener(this);\r
+        add.addClickHandler(this);\r
         remove = new VButton();\r
         remove.setText("<<");\r
-        remove.addClickListener(this);\r
+        remove.addClickHandler(this);\r
         panel = ((Panel) optionsContainer);\r
         panel.add(options);\r
         buttons.add(add);\r
@@ -143,9 +143,9 @@ public class VTwinColSelect extends VOptionGroupBase {
     }\r
 \r
     @Override\r
-    public void onClick(Widget sender) {\r
-        super.onClick(sender);\r
-        if (sender == add) {\r
+    public void onClick(ClickEvent event) {\r
+        super.onClick(event);\r
+        if (event.getSource() == add) {\r
             final boolean[] sel = getItemsToAdd();\r
             for (int i = 0; i < sel.length; i++) {\r
                 if (sel[i]) {\r
@@ -165,7 +165,7 @@ public class VTwinColSelect extends VOptionGroupBase {
             client.updateVariable(id, "selected", selectedKeys.toArray(),\r
                     isImmediate());\r
 \r
-        } else if (sender == remove) {\r
+        } else if (event.getSource() == remove) {\r
             final boolean[] sel = getItemsToRemove();\r
             for (int i = 0; i < sel.length; i++) {\r
                 if (sel[i]) {\r
@@ -183,13 +183,13 @@ public class VTwinColSelect extends VOptionGroupBase {
             }\r
             client.updateVariable(id, "selected", selectedKeys.toArray(),\r
                     isImmediate());\r
-        } else if (sender == options) {\r
+        } else if (event.getSource() == options) {\r
             // unselect all in other list, to avoid mistakes (i.e wrong button)\r
             final int c = selections.getItemCount();\r
             for (int i = 0; i < c; i++) {\r
                 selections.setItemSelected(i, false);\r
             }\r
-        } else if (sender == selections) {\r
+        } else if (event.getSource() == selections) {\r
             // unselect all in other list, to avoid mistakes (i.e wrong button)\r
             final int c = options.getItemCount();\r
             for (int i = 0; i < c; i++) {\r

diff --git a/src/com/vaadin/terminal/gwt/client/ui/richtextarea/VRichTextToolbar.java b/src/com/vaadin/terminal/gwt/client/ui/richtextarea/VRichTextToolbar.java
index 5325550b08434a53b74e716f856fbd49f6b6a819..b3220b4c7fd108aca095fa2152316bdb16be3493 100644 (file)
--- a/src/com/vaadin/terminal/gwt/client/ui/richtextarea/VRichTextToolbar.java
+++ b/src/com/vaadin/terminal/gwt/client/ui/richtextarea/VRichTextToolbar.java
@@ -1,12 +1,12 @@
 /*
  * Copyright 2007 Google Inc.
- * 
+ *
  * Licensed under the Apache License, Version 2.0 (the "License"); you may not
  * use this file except in compliance with the License. You may obtain a copy of
  * the License at
- * 
+ *
  * http://www.apache.org/licenses/LICENSE-2.0
- * 
+ *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
  * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
@@ -16,15 +16,17 @@
 package com.vaadin.terminal.gwt.client.ui.richtextarea;
 
 import com.google.gwt.core.client.GWT;
+import com.google.gwt.event.dom.client.ClickEvent;
+import com.google.gwt.event.dom.client.ClickHandler;
+import com.google.gwt.event.dom.client.KeyUpEvent;
+import com.google.gwt.event.dom.client.KeyUpHandler;
 import com.google.gwt.i18n.client.Constants;
 import com.google.gwt.user.client.Window;
 import com.google.gwt.user.client.ui.AbstractImagePrototype;
 import com.google.gwt.user.client.ui.ChangeListener;
-import com.google.gwt.user.client.ui.ClickListener;
 import com.google.gwt.user.client.ui.Composite;
 import com.google.gwt.user.client.ui.FlowPanel;
 import com.google.gwt.user.client.ui.ImageBundle;
-import com.google.gwt.user.client.ui.KeyboardListener;
 import com.google.gwt.user.client.ui.ListBox;
 import com.google.gwt.user.client.ui.PushButton;
 import com.google.gwt.user.client.ui.RichTextArea;
@@ -181,8 +183,8 @@ public class VRichTextToolbar extends Composite {
      * We use an inner EventListener class to avoid exposing event methods on
      * the RichTextToolbar itself.
      */
-    private class EventListener implements ClickListener, ChangeListener,
-            KeyboardListener {
+    private class EventListener implements ClickHandler, ChangeListener,
+            KeyUpHandler {
 
         public void onChange(Widget sender) {
             if (sender == backColors) {
@@ -203,7 +205,8 @@ public class VRichTextToolbar extends Composite {
             }
         }
 
-        public void onClick(Widget sender) {
+        public void onClick(ClickEvent event) {
+            Object sender = event.getSource();
             if (sender == bold) {
                 basic.toggleBold();
             } else if (sender == italic) {
@@ -259,17 +262,11 @@ public class VRichTextToolbar extends Composite {
             }
         }
 
-        public void onKeyDown(Widget sender, char keyCode, int modifiers) {
-        }
-
-        public void onKeyPress(Widget sender, char keyCode, int modifiers) {
-        }
-
-        public void onKeyUp(Widget sender, char keyCode, int modifiers) {
-            if (sender == richText) {
+        public void onKeyUp(KeyUpEvent event) {
+            if (event.getSource() == richText) {
                 // We use the RichTextArea's onKeyUp event to update the toolbar
                 // status.
-                // This will catch any cases where the user moves the cursur
+                // This will catch any cases where the user moves the cursor
                 // using the
                 // keyboard, or uses one of the browser's built-in keyboard
                 // shortcuts.
@@ -321,7 +318,7 @@ public class VRichTextToolbar extends Composite {
 
     /**
      * Creates a new toolbar that drives the given rich text area.
-     * 
+     *
      * @param richText
      *            the rich text area to be controlled
      */
@@ -388,8 +385,8 @@ public class VRichTextToolbar extends Composite {
             // We only use these listeners for updating status, so don't hook
             // them up
             // unless at least basic editing is supported.
-            richText.addKeyboardListener(listener);
-            richText.addClickListener(listener);
+            richText.addKeyUpHandler(listener);
+            richText.addClickHandler(listener);
         }
     }
 
@@ -442,7 +439,7 @@ public class VRichTextToolbar extends Composite {
 
     private PushButton createPushButton(AbstractImagePrototype img, String tip) {
         final PushButton pb = new PushButton(img.createImage());
-        pb.addClickListener(listener);
+        pb.addClickHandler(listener);
         pb.setTitle(tip);
         return pb;
     }
@@ -450,7 +447,7 @@ public class VRichTextToolbar extends Composite {
     private ToggleButton createToggleButton(AbstractImagePrototype img,
             String tip) {
         final ToggleButton tb = new ToggleButton(img.createImage());
-        tb.addClickListener(listener);
+        tb.addClickHandler(listener);
         tb.setTitle(tip);
         return tb;
     }