From 571792ed9ddb00ae795bd785473c181f1aa102a2 Mon Sep 17 00:00:00 2001 From: Leif Åstrand Date: Tue, 7 May 2013 15:45:51 +0300 Subject: Remove old widget hierarchy hack that caused problems (#11752) Change-Id: Icd66755512a2c1ab490e58f80389ed35966d3e3b --- client/src/com/vaadin/client/ui/VPopupView.java | 29 ---------------------- .../popupview/ClickingWhilePopupOpen.html | 2 +- 2 files changed, 1 insertion(+), 30 deletions(-) diff --git a/client/src/com/vaadin/client/ui/VPopupView.java b/client/src/com/vaadin/client/ui/VPopupView.java index d983da2b62..05fbd2c073 100644 --- a/client/src/com/vaadin/client/ui/VPopupView.java +++ b/client/src/com/vaadin/client/ui/VPopupView.java @@ -202,7 +202,6 @@ public class VPopupView extends HTML implements Iterable { private boolean hasHadMouseOver = false; private boolean hideOnMouseOut = true; private final Set activeChildren = new HashSet(); - private boolean hiding = false; private ShortcutActionHandler shortcutActionHandler; @@ -264,7 +263,6 @@ public class VPopupView extends HTML implements Iterable { @Override public void hide(boolean autoClosed) { VConsole.log("Hiding popupview"); - hiding = true; syncChildren(); if (popupComponentWidget != null && popupComponentWidget != loading) { remove(popupComponentWidget); @@ -276,8 +274,6 @@ public class VPopupView extends HTML implements Iterable { @Override public void show() { - hiding = false; - // Find the shortcut action handler that should handle keyboard // events from the popup. The events do not propagate automatically // because the popup is directly attached to the RootPanel. @@ -353,31 +349,6 @@ public class VPopupView extends HTML implements Iterable { this.hideOnMouseOut = hideOnMouseOut; } - /* - * - * 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 - public Widget getParent() { - if (!isAttached() || hiding) { - return super.getParent(); - } else { - return VPopupView.this; - } - } - - @Override - protected void onDetach() { - super.onDetach(); - hiding = false; - } - @Override public Element getContainerElement() { return super.getContainerElement(); diff --git a/uitest/src/com/vaadin/tests/components/popupview/ClickingWhilePopupOpen.html b/uitest/src/com/vaadin/tests/components/popupview/ClickingWhilePopupOpen.html index 81676f6599..43f1c2e8ed 100644 --- a/uitest/src/com/vaadin/tests/components/popupview/ClickingWhilePopupOpen.html +++ b/uitest/src/com/vaadin/tests/components/popupview/ClickingWhilePopupOpen.html @@ -28,7 +28,7 @@ assertTextNotPresent - IllegalStateException: Cannot set a new parent without first clearing the old parent + Uncaught client side exception -- cgit v1.2.3 en_updating_global_cred_in_files_external Nextcloud server, a safe home for all your data: https://github.com/nextcloud/serverwww-data
summaryrefslogtreecommitdiffstats
blob: 0117bbd83e304ef306a93c3ee88e176f84c8068c (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39