[CD-51] Close pop with gesture, add animation #32
Conversation
|
|
||
| @Override | ||
| public void onAnimationEnd(Animator animator) { | ||
|
|
| PropertyValuesHolder enterAnimationScaleY = PropertyValuesHolder.ofFloat("scaleY", EXIT_ICON_DEFAULT_SCALE_Y, EXIT_ICON_TARGET_SCALE_Y); | ||
| PropertyValuesHolder enterAnimationRotate = PropertyValuesHolder.ofFloat("rotation", EXIT_ICON_DEFAULT_ROTATION, EXIT_ICON_TARGET_ROTATION); | ||
| PropertyValuesHolder enterAnimationAlpha = PropertyValuesHolder.ofFloat("alpha", EXIT_ICON_DEFAULT_ALPHA, EXIT_ICON_TARGET_ALPHA); | ||
| enterAnimation = ObjectAnimator.ofPropertyValuesHolder(mExitIcon, enterAnimationScaleX, enterAnimationScaleY, enterAnimationRotate, enterAnimationAlpha); |
There was a problem hiding this comment.
전체적으로 this 를 붙이려다보니 가독성이 떨어지는 거 같아서, checkStyle 룰에 따라서 enterAnimation 을 private 으로 변경하고 m을 붙여 봤는데 어떤가요?
There was a problem hiding this comment.
저희 처음 할당할때 this를 붙여서 local variable이 아닌 member라고 명시하기로 했던것 같은데 예전에 했던 얘기라 기억이 잘 안나네요ㅋ m을 붙이면 리뷰볼때는 쉬울거 같긴한데 IDE에서 멤버들은 잘 표시를 해줘서 꼭 필요한건가 싶기도 하고!
There was a problem hiding this comment.
맞아요 this 를 붙이기로 했었는데 m을 붙인 이유는 checkStyle 이 private 변수일 때 m 을 강제해서 그렇게 적용이 됐어요
| @@ -71,4 +173,70 @@ private double calculateDistance(@NonNull Point p1, @NonNull Point p2) { | |||
| Math.pow(p2.x - p1.x, 2) + Math.pow(p2.y - p1.y, 2) | |||
| ); | |||
| } | |||
|
|
|||
| public void startEnterExitAnim() { | |||
| if (enterAnimation != null && !enterAnimation.isRunning() && !isExitAnimated) { | |||
There was a problem hiding this comment.
enterAnimation.isRunning만으로는 안되나요? isExitAnimated 이것도 필요한건가 해서요~
There was a problem hiding this comment.
isRunning 은 enterAnimation이 동작중인지 체크, isExitAnimated 는 현재 view 상태를 체크하는 건데, 가독성이 떨어지는 것 같아서 isExitAnimated -> isShowing 으로 변경했습니다
| @@ -71,4 +171,70 @@ private double calculateDistance(@NonNull Point p1, @NonNull Point p2) { | |||
| Math.pow(p2.x - p1.x, 2) + Math.pow(p2.y - p1.y, 2) | |||
| ); | |||
| } | |||
|
|
|||
| public void startShowAnimation() { | |||
There was a problem hiding this comment.
@toc2menow 리뷰해주신 부분 간단하게 show / hide 로 표현이 가능할 것 같아서 변경했는데 의미가 바뀌었네요. 수정후 다시 리뷰 요청드릴게요 😢
There was a problem hiding this comment.
@toc2menow start 를 없애고 show 로만 표현되도록 수정했습니다. showEnterAnimation, showExitAnimation
| * @return targetPoint | ||
| */ | ||
| private float getTargetYPosition(@NonNull Point point1, @NonNull Point point2) { | ||
| // STEP 1: get liner line equation from 2 points (ax + by = c) |
|
|
||
| return exitArea.contains(position.x, position.y) | ||
| && !excludedXExitAreaLeft.contains(position.x, position.y) | ||
| && !excludedXExitAreaRight.contains(position.x, position.y); | ||
| } | ||
|
|
||
| private Point getExitZoneCenter() { |
There was a problem hiding this comment.
isInExitZone method가 생기면서 사용을 안하는 것 같은데 없애도 되지 않을까요?
| @@ -71,4 +171,70 @@ private double calculateDistance(@NonNull Point p1, @NonNull Point p2) { | |||
| Math.pow(p2.x - p1.x, 2) + Math.pow(p2.y - p1.y, 2) | |||
| @@ -54,6 +55,10 @@ | |||
| private Handler mHandler = new Handler(); | |||
| private Runnable mIdleActionRunnable; | |||
| private Runnable mOnStateChanged; | |||
| private Point mStartPoint = null; | |||
| private long mDragStartMillis = -1L; | |||
There was a problem hiding this comment.
mStartPoint, mDragStartMillis 이 두개는 추가가 되었는데 사용되는 곳이 없네요.
나타날때, 사라질때, 종료영역 진입/이탈 시
좌/우 스와이프: Pop 위치 변경
아래로 스와이프: Pop 종료
TargetY 포지션은 x 가 0 or screen.width 일 때, 스와이프 한 방향과 만나는 지점으로 계산했습니다.