Documented why the order should be hierarchy listener before state

listener

diff --git a/src/com/vaadin/terminal/gwt/client/ApplicationConnection.java b/src/com/vaadin/terminal/gwt/client/ApplicationConnection.java
index 993221262367ea228fee054d5468a4577a685c50..fdaaf17d3b6ca342190a363725fe038c65390809 100644 (file)
--- a/src/com/vaadin/terminal/gwt/client/ApplicationConnection.java
+++ b/src/com/vaadin/terminal/gwt/client/ApplicationConnection.java
@@ -1013,12 +1013,16 @@ public class ApplicationConnection {
                 // Update hierarchy, do not fire events
                 Collection<ConnectorHierarchyChangedEvent> pendingHierarchyChangeEvents = updateConnectorHierarchy(json);
 
-                // Fire state change events (TODO)
-                VConsole.log(" * Sending state change events");
-
                 // Fire hierarchy change events
                 sendHierarchyChangeEvents(pendingHierarchyChangeEvents);
 
+                // (TODO) Fire state change events. Should be after hierarchy
+                // change listeners: At least caption updates for the parent are
+                // strange if fired from state change listeners and thus calls
+                // the parent BEFORE the parent is aware of the child (through a
+                // hierarchy change event)
+                VConsole.log(" * Sending state change events");
+
                 // Update of legacy (UIDL) style connectors
                 updateVaadin6StyleConnectors(json);