* [android] move element bug fix

alibaba · Nov 4, 2016 · dfc643d · dfc643d
1 parent 0a59a85
commit dfc643d
Showing 1 changed file with 11 additions and 5 deletions.
diff --git a/android/sdk/src/main/java/com/taobao/weex/dom/WXDomStatement.java b/android/sdk/src/main/java/com/taobao/weex/dom/WXDomStatement.java
@@ -716,7 +716,7 @@ void findFixed(WXDomObject obj){
    * @param parentRef Reference of the new parent DOM node
    * @param index the index of the dom to be inserted in the new parent.
    */
-  void moveDom(final String ref, final String parentRef, final int index) {
+  void moveDom(final String ref, final String parentRef, int index) {
     if (mDestroy) {
       return;
     }
@@ -730,17 +730,23 @@ void moveDom(final String ref, final String parentRef, final int index) {
       }
       return;
     }
-    if (domObject.parent.equals(parentObject) && parentObject.index(domObject) == index) {
-      return;
+    if (domObject.parent.equals(parentObject)) {
+      if(parentObject.index(domObject) == index) {
+        return;
+      } else if(domObject.parent.index(domObject)<index){
+        index = index -1;
+      }
     }
+
+    final int newIndex = index;
     domObject.parent.remove(domObject);
-    parentObject.add(domObject, index);
+    parentObject.add(domObject, newIndex);
 
     mNormalTasks.add(new IWXRenderTask() {
 
       @Override
       public void execute() {
-        mWXRenderManager.moveComponent(mInstanceId, ref, parentRef, index);
+        mWXRenderManager.moveComponent(mInstanceId, ref, parentRef, newIndex);
       }
 
       @Override