feat: add scroll action in ProfileFragment

[HeewonP825]

Related Issue

#25

Changes

작업 사항

• ProfileFragment의 전체 화면 ScrollView 구현 완료
  • ScrollView 와 RecyclerView, TabLayout의 중첩 스크롤 문제로 인해 NestedScrollView 사용
  • ViewPager2 제거 후 프로필 페이지에서 직접 RecyclerView 사용
  • SwipeRefresh 추가
  • ProfileTapRvDataHelper로 메서드 추출/분리 완료
• backbtn에 TimeLineFragment로 이동하는 action 추가

수정 사항

• Theme/colors 파일에 정의된 색상 테마 추가 및 수정
  • Red->Pink 및 Red 추가

Screenshots

작업화면

To Reviewer

TODO

• Highlight 탭에서 보여주는 유료구독페이지는 따로 RV로 만들어서 추후 적용해보겠습니당...로직이 좀 꼬이네요ㅜ
• 스크롤 시 header 및 TabLayout 고정하는 부분은 에디터 만든 이후에 만들겠습니당,,ㅎㅎㅎㅎ

Checklist

• PR 제목은 포맷과 내용 둘 다 알맞게 작성되었는가
• PR에 대해 구체적으로 설명이 되어있는가

[poiu694]

리뷰 남겼습니다 ! 고생하셨어요 :)

[poiu694]

eol :)

[HeewonP825]

551ee38 eol 모두 처리해서 반영했습니다!

[poiu694]

여기는 추후라도 사용하게 되는 코드인가요?

[HeewonP825]

swipe action을 통해 데이터를 리프레쉬 시킬 때 사용할 함수인데 메서드 추출로 인해 클래스를 분리한 상태입니다! ProfileFragment의 코드에 이 함수를 사용하는 부분이 있고 추후 binding 부분을 바꾸어 처리할 예정이라 내용만 주석처리해두었습니다:)

[poiu694]

죠습니다 👍

[poiu694]

해당 주석은 어떤 내용인가요?

[HeewonP825]

Highlight 탭에서 보여주는 유료구독페이지를 예전처럼 ViewPager2가 아닌 RecyclerView로 보여줘야 하기때문에 Data Class 와 Adapter를 만들어서 보여주는 방법을 시도해본건데 GPT한테 부탁한 코드를 그대로 복사해서 주석을 안지웠네용..ㅎㅎ 지웠습니다!!ㅎㅎㅎ

[poiu694]

죠습니다~!!

[poiu694]

eol :)

[HeewonP825]

551ee38 eol 모두 처리해서 반영했습니다!

[poiu694]

해당 부분도 if가 너무 중첩이라서요! 개선해 보면 어떨까요?

fun moveAction(x: Number, y: Number) {
    val dx = x - initialX
    val dy = y - initialY
    val isVpHorizontal = orientation == ORIENTATION_HORIZONTAL

    // assuming ViewPager2 touch-slop is 2x touch-slop of child
    val scaledDx = dx.absoluteValue * if (isVpHorizontal) .5f else 1f
    val scaledDy = dy.absoluteValue * if (isVpHorizontal) 1f else .5f

    if (scaledDx <= touchSlop && scaledDy <= touchSlop)  {
        return
    }

    val isVerticalGesture = scaledDy > scaledDx
    if (isVpHorizontal == isVerticalGesture)  {
        // Gesture is perpendicular, allow all parents to intercept
        parent.requestDisallowInterceptTouchEvent(false)
        return
    }
    parent.requestDisallowInterceptTouchEvent(canChildScroll(orientation, if (isVpHorizontal) dx else dy)
}

if (e.action == MotionEvent.ACTION_DOWN) {
    downAction(e.x, e.y)
}
if (e.action == MotionEvent.ACTION_MOVE) {
    moveAction(e.x, e.y)
}

이런 느낌으로도 될까요?

[HeewonP825]

원래 ViewPager2를 사용하면서 여기정기 방법을 찾아서 구글링 하던 도중에 NestedScrollView를 사용하기 위해서 이 클래스를 레포에 넣으라고 해서 넣었던 코드인데.. 결과적으로 그 코드를 사용하지 않게되었는데 깜빡하고 지우지를 못했네요ㅜ 삭제하고 각 fragment_xxx.xml에서 사용했던 부분도 삭제 처리 완료했습니다!:) 8a45fab

[poiu694]

좋습니당 :)

[poiu694]

여기도 아래처럼 하면 좋을 것 같아요 :)

if (e1 == null || e2 == null)
    return false
val deltaX = Math.abs(e2.x - e1.x)
val deltaY = Math.abs(e2.y - e1.y)

if (deltaX < deltaY)
    return false
if (Math.abs(deltaX) > SWIPE_THRESHOLD && Math.abs(velocityX) > SWIPE_VELOCITY_THRESHOLD) {
     if (deltaX > 0) {
         // 오른쪽으로 스와이프한 경우
         navigateToPreviousTab()
     } else {
         // 왼쪽으로 스와이프한 경우
         navigateToNextTab()
     }
     return true
 }

[HeewonP825]

ff578c2 이 부분도 최대한 메서드로 분리해서 리뷰해주신 부분 반영 완료했습니다ㅎㅎ

gestureDetectorTab = GestureDetector(requireContext(), object : GestureDetector.SimpleOnGestureListener() {
            override fun onFling(
                e1: MotionEvent?,
                e2: MotionEvent,
                velocityX: Float,
                velocityY: Float
            ): Boolean {
                return handleSwipe(e1, e2, velocityX)
            }
        })

fun handleSwipe(e1: MotionEvent?, e2: MotionEvent?, velocityX: Float): Boolean {
        if (e1 != null && e2 != null) {
            val deltaX = e2.x - e1.x
            val deltaY = e2.y - e1.y
            return handleHorizontalSwipe(deltaX, deltaY, velocityX)
        }
        return false
    }

    private fun handleHorizontalSwipe(deltaX: Float, deltaY: Float, velocityX: Float): Boolean {
        val absDeltaX = abs(deltaX)
        val absDeltaY = abs(deltaY)

        if (absDeltaX > absDeltaY && absDeltaX > SWIPE_THRESHOLD && Math.abs(velocityX) > SWIPE_VELOCITY_THRESHOLD) {
            if (deltaX > 0) {
                // 오른쪽으로 스와이프한 경우
                navigateToPreviousTab()
            } else {
                // 왼쪽으로 스와이프한 경우
                navigateToNextTab()
            }
            return true
        }
        return false
    }

[poiu694]

아래처럼 작성하면 if문 다이어트를 할 수 있지 않을까요?

tabLayout.getTabAt(Math.min(currentTab + 1, tabTitles.size - 1))?.select()

[poiu694]

아래처럼 작성하면 if문 다이어트를 할 수 있지 않을까요? 22

tabLayout.getTabAt(Math.max(currentTab - 1, 0))?.select()

[HeewonP825]

tabLayout.getTabAt((currentTab - 1).coerceAtLeast(0))?.select()

반영완료했습니다!

[poiu694]

상수화 👍

[poiu694]

eol :)

[poiu694]

고생하셨습니다 👍