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

[UI/#21] 성향테스트뷰 UI / Animation 구현 #23

Merged
merged 18 commits into from
Jan 4, 2024

Conversation

chattymin
Copy link
Member

@chattymin chattymin commented Jan 4, 2024

⛳️ Work Description

  • 툴바 구현
  • 프로그래스바 구현
  • 진행 숫자표시(1/9) 및 질문 구현
  • 질문에 대한 응답 버튼 구현
  • 다음 버튼 구현
  • 애니메이션 효과 구현
  • 데이터 모델 및 mock데이터 삽입

📸 Screenshot

2024-01-04.7.00.35.mov

📢 To Reviewers

  • 처음해보는 기술을 쓰고 커스텀을 계속 하다보니 코드가 너무 더러운거 같아요...
  • 수정해야 할 부분이나 최적화 방법은 언제나 환영입니다!!
  • progressbar를 큰 숫자로 만들고 1씩 늘리는 방법으로 애니메이션을 했는데 성능에도 영향을 줄 것 같아서 고민이네요... 물론 성능이 중요한 페이지는 아니긴 하지만 ㅋㅋ

어라 폰트 하나 안넣었네요! 빠르게 수정해서 올릴테니 그 부분 제외하고 코리 달아주세용~
-> 폰트 적용 완료

Copy link
Member

@Marchbreeze Marchbreeze left a comment

Choose a reason for hiding this comment

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

아니 걍 말도안됨 건드릴게 없어

Comment on lines +48 to +50
<activity
android:name="com.going.presentation.tendencytest.TendencyTestSplashActivity"
android:exported="false" />
Copy link
Member

Choose a reason for hiding this comment

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

목 액티비티처럼 화면 회전 막아주세용 ~

Copy link
Member Author

Choose a reason for hiding this comment

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

이건 몰랐었네요!
이전에 제가 만든 액티비티들에도 다 적용했습니다 :)

binding.btnTendencySplashStart.setOnSingleClickListener {
// 페이지 이동~
private fun setProgressAnimate(pb: ProgressBar, progressTo: Int) =
ObjectAnimator.ofInt(pb, "progress", pb.progress, progressTo * 100).apply {
Copy link
Member

Choose a reason for hiding this comment

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

"progress" 같은 친구들도 const val로 표시해두면 좋을듯요!

Copy link
Member Author

Choose a reason for hiding this comment

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

아 깜빡하고 안했네요!! 수정 완료했습니다
아주 조아용~

initStartBtnSingleClickListener()
}

private fun initStartBtnSingleClickListener() {
Copy link
Member

Choose a reason for hiding this comment

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

싱글클릭 말고 다른 리스너 달거 아니면 initStartBtnClickListener로 수정해주셍요 ~

Copy link
Member Author

Choose a reason for hiding this comment

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

이게 더 좋은 방법이네요!
매번 적으려 했던 박동민 바보...

@chattymin chattymin merged commit 0d80ecc into develop Jan 4, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[UI] 성향테스트 뷰 / UI 구현
2 participants