修复部分bug

chiemy · chiemy · commit 5974d3fe04ce · 2014-09-11T00:19:30.000+08:00
diff --git a/readme.md b/readme.md
@@ -6,6 +6,13 @@
 
 - 测试版本4.1.2，4.2.2，4.4.2可用
 
+##2014-9-11修改
+
+- 修复设置高度`wrap_content`时显示不全的bug
+- 修复点击后，翻转过程中向下滑动出现的bug
+
+>注意：如果设置高度为`wrap_content`，为了保证动画视觉上的完整性，需要设置自身及所有关联窗口的`android:clipChildren`属性为false(代码中setClipChildren(false))。
+
 ##使用
 
 
@@ -23,7 +30,5 @@
 
 ##问题
 
-- `CardView`高度设置为`wrap_content`卡片会显示不全。(已修复)
-- `CardView`设置`padding`和`margin`，旋转动画会有被截断的效果。最后一张卡片距离顶部的距离是在CardView内部指定的，为固定值。
 - 测试2.3.2有严重bug（估计4.0版本之前都有此bug），不可用。
 
diff --git a/res/layout/activity_main.xml b/res/layout/activity_main.xml
@@ -4,12 +4,13 @@
     android:layout_height="match_parent"
     tools:context="com.chiemy.cardview.MainActivity"
     android:background="#00688B"
+    android:clipChildren="false"
      >
 
     <com.chiemy.cardview.view.CardView
         android:id="@+id/cardView1"
         android:layout_width="match_parent"
-        android:layout_height="match_parent"
+        android:layout_height="wrap_content"
         >
     </com.chiemy.cardview.view.CardView>
 	
diff --git a/src/com/chiemy/cardview/view/CardView.java b/src/com/chiemy/cardview/view/CardView.java
@@ -4,13 +4,10 @@
 import android.database.DataSetObserver;
 import android.graphics.Rect;
 import android.util.AttributeSet;
-import android.util.Log;
 import android.util.SparseArray;
 import android.view.MotionEvent;
 import android.view.View;
 import android.view.ViewConfiguration;
-import android.view.ViewGroup;
-import android.view.View.MeasureSpec;
 import android.view.animation.AccelerateInterpolator;
 import android.widget.FrameLayout;
 import android.widget.ListAdapter;
@@ -107,7 +104,6 @@ && getChildCount() < mMaxVisible) {
 			final View view = mListAdapter.getView(mNextAdapterPosition,
 					convertView, this);
 			view.setOnClickListener(null);
-			// view.setOnTouchListener(null);
 			viewHolder.put(index, view);
 
 			// 添加剩余的View时，始终处在最后
@@ -173,17 +169,13 @@ protected void onLayout(boolean changed, int left, int top, int right,
 	}
 
 	float downX, downY;
-	boolean remove = false;
 
 	@Override
 	public boolean onTouchEvent(MotionEvent event) {
 		switch (event.getAction()) {
 		case MotionEvent.ACTION_MOVE:
-			if (!remove) {
-				if (goDown()) {
-					downY = -1;
-					remove = true;
-				}
+			if (goDown()) {
+				downY = -1;
 			}
 			break;
 		}
@@ -195,12 +187,16 @@ public boolean onTouchEvent(MotionEvent event) {
 	 */
 	private boolean goDown() {
 		final View topView = getChildAt(getChildCount() - 1);
+		if(!topView.isClickable()){
+			return false;
+		}
 		// topView.getHitRect(topRect); 在4.3以前有bug，用以下方法代替
 		topRect = getHitRect(topRect, topView);
 		// 如果按下的位置不在顶部视图上，则不移动
 		if (!topRect.contains((int) downX, (int) downY)) {
 			return false;
 		}
+		topView.setClickable(false);
 		ViewPropertyAnimator anim = ViewPropertyAnimator
 				.animate(topView)
 				.translationY(
@@ -210,6 +206,7 @@ private boolean goDown() {
 		anim.setListener(new AnimatorListenerAdapter() {
 			@Override
 			public void onAnimationEnd(Animator animation) {
+				topView.setClickable(true);
 				removeView(topView);
 				ensureFull();
 				final int count = getChildCount();
@@ -245,6 +242,7 @@ public void onAnimationEnd(Animator animation) {
 	 * @param view
 	 */
 	private void bringToTop(final View view) {
+		topPosition++;
 		float scaleX = ViewHelper.getScaleX(view) + ((float) 1 / mMaxVisible)
 				* 0.2f;
 		float tranlateY = ViewHelper.getTranslationY(view) + itemSpace;
@@ -253,8 +251,7 @@ private void bringToTop(final View view) {
 				.setInterpolator(new AccelerateInterpolator())
 				.setListener(null).setListener(new AnimatorListenerAdapter() {
 					public void onAnimationEnd(Animator animation) {
-						topPosition++;
-						remove = false;
+						
 					}
 				});
 	}

Original file line number	Diff line number	Diff line change
`@@ -4,12 +4,13 @@`
`4`	`4`	`android:layout_height="match_parent"`
`5`	`5`	`tools:context="com.chiemy.cardview.MainActivity"`
`6`	`6`	`android:background="#00688B"`
	`7`	`+ android:clipChildren="false"`
`7`	`8`	`>`
`8`	`9`
`9`	`10`	`<com.chiemy.cardview.view.CardView`
`10`	`11`	`android:id="@+id/cardView1"`
`11`	`12`	`android:layout_width="match_parent"`
`12`		`- android:layout_height="match_parent"`
	`13`	`+ android:layout_height="wrap_content"`
`13`	`14`	`>`
`14`	`15`	`</com.chiemy.cardview.view.CardView>`
`15`	`16`