From 8a107f54b24e7f3934306483f7928360490f3eb0 Mon Sep 17 00:00:00 2001 From: John Alhroos Date: Fri, 4 Mar 2011 12:13:07 +0000 Subject: [PATCH] Fixes issue with double clicks when browsing ComboBox pages #4285 svn changeset:17608/svn branch:6.5 --- .../terminal/gwt/client/ui/VFilterSelect.java | 16 +++++++++------- 1 file changed, 9 insertions(+), 7 deletions(-) diff --git a/src/com/vaadin/terminal/gwt/client/ui/VFilterSelect.java b/src/com/vaadin/terminal/gwt/client/ui/VFilterSelect.java index 7c1a4b2837..f9d9f8dbc9 100644 --- a/src/com/vaadin/terminal/gwt/client/ui/VFilterSelect.java +++ b/src/com/vaadin/terminal/gwt/client/ui/VFilterSelect.java @@ -326,13 +326,15 @@ public class VFilterSelect extends Composite implements Paintable, Field, */ @Override public void onBrowserEvent(Event event) { - final Element target = DOM.eventGetTarget(event); - if (DOM.compare(target, up) - || DOM.compare(target, DOM.getChild(up, 0))) { - filterOptions(currentPage - 1, lastFilter); - } else if (DOM.compare(target, down) - || DOM.compare(target, DOM.getChild(down, 0))) { - filterOptions(currentPage + 1, lastFilter); + if (event.getTypeInt() == Event.ONCLICK) { + final Element target = DOM.eventGetTarget(event); + if (DOM.compare(target, up) + || DOM.compare(target, DOM.getChild(up, 0))) { + filterOptions(currentPage - 1, lastFilter); + } else if (DOM.compare(target, down) + || DOM.compare(target, DOM.getChild(down, 0))) { + filterOptions(currentPage + 1, lastFilter); + } } /* -- 2.39.5