Handle replacement of widgets, it needs to deal with more gwt panels

diff --git a/gwtquery-core/src/main/java/com/google/gwt/query/client/plugins/Widgets.java b/gwtquery-core/src/main/java/com/google/gwt/query/client/plugins/Widgets.java
index bc3f35e8ae38702b4824c034390ff8cc400c4f56..4e452ff51c5a341be2451ce08fc5f8821cb819ba 100755 (executable)
--- a/gwtquery-core/src/main/java/com/google/gwt/query/client/plugins/Widgets.java
+++ b/gwtquery-core/src/main/java/com/google/gwt/query/client/plugins/Widgets.java
@@ -446,7 +446,7 @@ public class Widgets extends QueuePlugin<Widgets> {
   }\r
 \r
   protected boolean isWidgetCreationAuthorizedFrom(Element e) {\r
-    return $(e).widget() == null && !WidgetsUtils.matchesTags(e, excludedTags);\r
+    return !WidgetsUtils.matchesTags(e, excludedTags);\r
   }\r
 \r
   /**\r
@@ -461,9 +461,7 @@ public class Widgets extends QueuePlugin<Widgets> {
 \r
     W widget = factory.create(e);\r
     if (initializer != null) {\r
-\r
       initializer.initialize(widget, e);\r
-\r
     }\r
     return widget;\r
   }\r

diff --git a/gwtquery-core/src/main/java/com/google/gwt/query/client/plugins/widgets/WidgetsUtils.java b/gwtquery-core/src/main/java/com/google/gwt/query/client/plugins/widgets/WidgetsUtils.java
index ef247fac348223d65f23f07d9b9cbd9efc8f77da..a5b0c5e52c3ebbc97d019cb60b161de53409095b 100644 (file)
--- a/gwtquery-core/src/main/java/com/google/gwt/query/client/plugins/widgets/WidgetsUtils.java
+++ b/gwtquery-core/src/main/java/com/google/gwt/query/client/plugins/widgets/WidgetsUtils.java
@@ -15,9 +15,15 @@
  */
 package com.google.gwt.query.client.plugins.widgets;
 
+import static com.google.gwt.query.client.GQuery.$;
+
 import com.google.gwt.dom.client.Element;
 import com.google.gwt.query.client.GQuery;
+import com.google.gwt.user.client.ui.ComplexPanel;
 import com.google.gwt.user.client.ui.GqUi;
+import com.google.gwt.user.client.ui.HTMLPanel;
+import com.google.gwt.user.client.ui.Panel;
+import com.google.gwt.user.client.ui.SimplePanel;
 import com.google.gwt.user.client.ui.Widget;
 
 public class WidgetsUtils {
@@ -76,15 +82,35 @@ public class WidgetsUtils {
     }
    }
    
-   private static void hideAndAppend(Element oldElement, Element newElement) {
+   private static void hideAndAfter(Element oldElement, Element newElement) {
      assert oldElement != null && newElement != null;
      GQuery.$(oldElement).hide().after(newElement);
      String c = oldElement.getClassName();
      if (!c.isEmpty()) {
        newElement.addClassName(c);
      }
-    }
-
+   }
+   
+   private static void replaceWidget(Widget oldWidget, Widget newWidget, boolean remove) {
+     Widget parent = oldWidget.getParent();
+     // TODO: handle tables
+     if (parent instanceof HTMLPanel) {
+       ((HTMLPanel) parent).addAndReplaceElement(newWidget, oldWidget.getElement().<com.google.gwt.dom.client.Element>cast());
+     } else if (parent instanceof ComplexPanel) {
+       ((ComplexPanel) parent).add(newWidget);
+     } else if (parent instanceof SimplePanel) {
+       ((SimplePanel) parent).setWidget(newWidget);
+     } else if (parent instanceof Panel) {
+       ((Panel) parent).add(newWidget);
+     } else {
+       assert false : "Can not replace an attached widget whose parent is a " + parent.getClass().getName();
+     }
+     if (remove) {
+       oldWidget.removeFromParent();
+     } else {
+       oldWidget.setVisible(false);
+     }
+   }
    
    /**
     * Replace a dom element by a widget.
@@ -92,20 +118,27 @@ public class WidgetsUtils {
     */
    public static void replaceOrAppend(Element e, Widget widget)  {
      assert e != null && widget != null;
-     GqUi.detachWidget(widget);
-     replaceOrAppend(e, widget.getElement());
-     GqUi.attachWidget(widget);
+     if ($(e).widget() != null) {
+       replaceWidget($(e).widget(), widget, true);
+     } else {
+       GqUi.detachWidget(widget);
+       replaceOrAppend(e, widget.getElement());
+       GqUi.attachWidget(widget);
+     }
    }
 
    /**
     * Append a widget to a dom element, and hide it.
     * Element classes will be copied to the new widget.
     */
-   public static void hideAndAppend(Element e, Widget widget)  {
+   public static void hideAndAfter(Element e, Widget widget)  {
      assert e != null && widget != null;
-     GqUi.detachWidget(widget);
-     hideAndAppend(e, widget.getElement());
-     GqUi.attachWidget(widget);
+     if ($(e).widget() != null) {
+       replaceWidget($(e).widget(), widget, false);
+     } else {
+       GqUi.detachWidget(widget);
+       hideAndAfter(e, widget.getElement());
+       GqUi.attachWidget(widget);
+     }
    }
-  
 }