From: Johannes Dahlström <johannesd@vaadin.com>
Date: Fri, 19 Oct 2012 15:21:46 +0000 (+0300)
Subject: Register CssLayout click listeners in onStateChanged instead of onConnectorHierarchyC... 
X-Git-Tag: 7.0.0.beta6~20^2
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=refs%2Fchanges%2F42%2F142%2F1;p=vaadin-framework.git

Register CssLayout click listeners in onStateChanged instead of onConnectorHierarchyChange (#10003)

Test case (tests.components.ClickListeners) committed as part of #9956

Change-Id: I35f41a301ceebc43679098ae033656a18b82c4b1
---

diff --git a/client/src/com/vaadin/client/ui/csslayout/CssLayoutConnector.java b/client/src/com/vaadin/client/ui/csslayout/CssLayoutConnector.java
index 12ebb6a375..cfdda3dd1c 100644
--- a/client/src/com/vaadin/client/ui/csslayout/CssLayoutConnector.java
+++ b/client/src/com/vaadin/client/ui/csslayout/CssLayoutConnector.java
@@ -93,6 +93,7 @@ public class CssLayoutConnector extends AbstractLayoutConnector {
     @Override
     public void onStateChanged(StateChangeEvent stateChangeEvent) {
         super.onStateChanged(stateChangeEvent);
+        clickEventHandler.handleEventHandlerRegistration();
 
         for (ComponentConnector child : getChildComponents()) {
             if (!getState().childCss.containsKey(child)) {
@@ -123,8 +124,6 @@ public class CssLayoutConnector extends AbstractLayoutConnector {
      */
     @Override
     public void onConnectorHierarchyChange(ConnectorHierarchyChangeEvent event) {
-        clickEventHandler.handleEventHandlerRegistration();
-
         int index = 0;
         for (ComponentConnector child : getChildComponents()) {
             VCaption childCaption = childToCaption.get(child);