Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Merge] #14 미팅룸 카드 좌우스크롤 & 롱프레스드래그 애니메이션 구현 #54

Merged
merged 1 commit into from Jun 11, 2022

Conversation

ccdkk
Copy link
Collaborator

@ccdkk ccdkk commented Jun 11, 2022

작업 내용

  • Dake가 작업한 미팅룸 하단 카드뷰 좌우스크롤과 Dani가 작업한 롱프레스 & 상하드래그 애니메이션 합쳤습니다.

리뷰 포인트

  • 변수 선언이 직관적인인지
  • 애니메이션 자연스러운지

다음으로 진행될 작업

  • 주석정리 & 코드 정리

질문

  • CGFloat을 모두 화면비율로 전환할까요?
  • function을 한 곳에 때려박다보니 코드가 너무 길어지고지는 것 같아서, 효율적이고 가시적으로 func를 관리할 수 있는 방법이 궁금합니다!

References

- 다니가 보내준 롱프레스드래그 애니메이션과 좌우스크롤 연동

Ref: 14
Copy link
Collaborator

@YeeunKim-archive YeeunKim-archive left a comment

Choose a reason for hiding this comment

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

확인했습니다.

@YeeunKim-archive YeeunKim-archive merged commit 954245c into develop Jun 11, 2022
@YeeunKim-archive
Copy link
Collaborator

enum은 따로 모아서 관리할 수 있을 것 같아요. function 분리는 저도 궁금합니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants