Fix bauerca#59: dynamic list item heights

Set DragSortItemView layout param height to WRAP_CONTENT when it is not a special expanded/collapsed item. This way, ListView creates the appropriate height MeasureSpec while laying out items.
xyz6382 · Jan 13, 2013 · 6afde13 · 6afde13
1 parent ec55b6f
commit 6afde13
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/library/src/com/mobeta/android/dslv/DragSortItemView.java b/library/src/com/mobeta/android/dslv/DragSortItemView.java
@@ -36,7 +36,7 @@ public DragSortItemView(Context context) {
                 ViewGroup.LayoutParams.FILL_PARENT,
                 ViewGroup.LayoutParams.WRAP_CONTENT));
 
-        setClipChildren(true);
+        //setClipChildren(true);
     }
 
     public void setGravity(int gravity) {

diff --git a/library/src/com/mobeta/android/dslv/DragSortListView.java b/library/src/com/mobeta/android/dslv/DragSortListView.java
@@ -1729,17 +1729,20 @@ private void adjustItem(int position) {
     private void adjustItem(int position, View v, boolean invalidChildHeight) {
 
         // Adjust item height
-
-        int height = calcItemHeight(position, v, invalidChildHeight);
         ViewGroup.LayoutParams lp = v.getLayoutParams();
+        int height;
+        if (position != mSrcPos && position != mFirstExpPos && position != mSecondExpPos) {
+            height = ViewGroup.LayoutParams.WRAP_CONTENT;
+        } else {
+            height = calcItemHeight(position, v, invalidChildHeight);
+        }
 
         if (height != lp.height) {
             lp.height = height;
             v.setLayoutParams(lp);
         }
 
         // Adjust item gravity
-
         if (position == mFirstExpPos || position == mSecondExpPos) {
             if (position < mSrcPos) {
                 ((DragSortItemView) v).setGravity(Gravity.BOTTOM);