Skip to content
This repository has been archived by the owner on Feb 9, 2023. It is now read-only.

[CD-91] Fix disappearing pop icon #34

Merged
merged 6 commits into from
Sep 25, 2019

Conversation

realwind2048
Copy link

팝을 Throw 해서 종료 애니메이션 중에, showPop을 할 경우 팝아이콘이 사라지는 문제를 수정

  • Disappear 애니메이션 중인데, 이를 감지하지 못하고, 그냥 show 상태로 인식한 것이 문제
  • show 상태에서도 명시적으로 Appear (without animation)를 수행하도록 변경
  • disappearing 애니메이션이 실행 중이면 중단하는 코드도 추가

@realwind2048
Copy link
Author

버그가 발견되어 잠시 닫을게요 고치고 리오픈하겠습니다 😢

@realwind2048 realwind2048 reopened this Sep 25, 2019
@@ -303,7 +304,11 @@ private void closeWithThrowingAnimation(Point targetPoint) {
mFloatingTab.closeAnimation(targetPoint, new Runnable() {
@Override
public void run() {
activateDragger();
try {
activateDragger();
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@josh-yun 일반적인 상황에서는 발생하지 않을 것 같아서 try catch 로 잡아봤습니다.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@realwind2048 activateDragger() 함수에서 null check 를 하는 방식은 이상할까요? State가 가지고 있는 mHoverView의 경우 null 일 경우도 있는 것 같아요. (Ex. State 전환) 만약 null check 방식으로 바뀔 경우, activate에 실패하면 앱이 죽는 대신 클릭이 안되는 상황이 발생할텐데... 어떤 것이 더 critical 할 지는 잘 모르겠지만, Pop을 끄는 상황에서는 다른점이 없어보이네요.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

try catch 보다 그 쪽이 명확할 것 같아서 반영했습니다

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@josh-yun 943672d 에서 이상한곳에 적용돼서.. 2748736 에서 다시 수정했습니다 ㅎㅎ;; 이미 적용한 것도 방어코드라 놔뒀습니다.

This reverts commit 4f49a5a
Copy link

@toc2menow toc2menow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

에니메이션이 들어가니 좀 복잡해지긴 하네요 LGTM:)

Copy link

@josh-yun josh-yun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@realwind2048 realwind2048 merged commit 624d2a0 into fix/pop_throw_close Sep 25, 2019
@delete-merged-branch delete-merged-branch bot deleted the fix/disppearing_pop_icon branch September 25, 2019 08:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants