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

[AN/USER] feat: 애니메이션 수정 축제 로딩 개선(#876) #882

Merged
merged 5 commits into from
Apr 24, 2024

Conversation

SeongHoonC
Copy link
Member

📌 관련 이슈

✨ PR 세부 내용

커스텀 애니메이션 적용

  • slide_in : 오른쪽에서 왼쪽으로 슬라이드로 등장하면서 서서히 나타난다
  • slide_out : 왼쪽에서 오른쪽으로 슬라이드 되면서 서서히 사라진다.
  • no_anim : 그대로 유지한다. (설정을 안해주니깐 빈 화면이 되어버려서 추가했습니다.)
  • fade_in : 서서히 나타난다.
  • fade_out: 서서히 사라진다.

상세화면으로 들어가는 것들은 모두 slide 를 사용했습니다.
검색 화면은 fade 로 들어갑니다.

페이징 방식 변경

  1. 기존 방식
    스크롤 이벤트가 발생했을 때 리사이클러뷰 바닥에 도달하면 요청
    바닥에 도달해야지만 재요청하기 때문에 문제 발생
  2. FestivalMoreItemUiState (Progress Bar View) 의 바인드 이용
    바인드되는 시점에 람다 함수를 실행. 바인드는 실제로 보이기 이전에 발생하기 때문에 보다 자연스러운 스크롤이 가능.
    물론 겁나 빨리 스크롤하면 끊김.

@SeongHoonC SeongHoonC added AN 안드로이드에 관련된 작업 USER labels Apr 20, 2024
@SeongHoonC SeongHoonC self-assigned this Apr 20, 2024
@SeongHoonC SeongHoonC linked an issue Apr 20, 2024 that may be closed by this pull request
2 tasks
@github-actions github-actions bot requested review from EmilyCh0 and re4rk April 20, 2024 07:00
Copy link
Member

@EmilyCh0 EmilyCh0 left a comment

Choose a reason for hiding this comment

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

화면 전환 애니메이션 훨씬 자연스러워졌네요 👍👍

Copy link
Collaborator

@re4rk re4rk left a comment

Choose a reason for hiding this comment

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

LGTM!

@SeongHoonC SeongHoonC merged commit 7c78672 into dev Apr 24, 2024
1 check passed
@SeongHoonC SeongHoonC deleted the feat/#876 branch April 24, 2024 02:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AN 안드로이드에 관련된 작업 USER
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[AN] 애니메이션을 적용한다.
3 participants