From d295e5a71af8c9281c1982f79bb96ec832729966 Mon Sep 17 00:00:00 2001 From: John Alhroos Date: Fri, 4 Mar 2011 15:39:17 +0000 Subject: Fix for #4285 : Blur event prevention mechanism for IE prevented focus from moving to another field when first focusing the field, then opening the popup dialog and the clicking on another field. This fixed by only preventing the blur event if we already have focus. svn changeset:17623/svn branch:6.5 --- src/com/vaadin/terminal/gwt/client/ui/VFilterSelect.java | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) (limited to 'src/com/vaadin') diff --git a/src/com/vaadin/terminal/gwt/client/ui/VFilterSelect.java b/src/com/vaadin/terminal/gwt/client/ui/VFilterSelect.java index da5f879315..66adc39d63 100644 --- a/src/com/vaadin/terminal/gwt/client/ui/VFilterSelect.java +++ b/src/com/vaadin/terminal/gwt/client/ui/VFilterSelect.java @@ -730,7 +730,6 @@ public class VFilterSelect extends Composite implements Paintable, Field, /** * Used when measuring the width of the popup */ - private final HTML popupOpener = new HTML("") { /* @@ -1709,8 +1708,11 @@ public class VFilterSelect extends Composite implements Paintable, Field, /* * In IE the above wont work, the blur event will still trigger. So, * we set a flag here to prevent the next blur event from happening. + * This is not needed if do not already have focus, in that case + * there will not be any blur event and we should not cancel the + * next blur. */ - if (BrowserInfo.get().isIE()) { + if (BrowserInfo.get().isIE() && focused) { preventNextBlurEventInIE = true; } } -- cgit v1.2.3