Skip to content

인증샷 상세 당겨서 새로고침 추가#172

Open
chanho0908 wants to merge 5 commits into
developfrom
feature/#169-photolog-detail-refresh
Open

인증샷 상세 당겨서 새로고침 추가#172
chanho0908 wants to merge 5 commits into
developfrom
feature/#169-photolog-detail-refresh

Conversation

@chanho0908

@chanho0908 chanho0908 commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

이슈 번호

#169

작업내용

  • 인증샷 상세 화면에 당겨서 새로고침 인터랙션을 추가
  • 사용자 새로고침 중에는 전체 오버레이 로딩 대신 인디케이터가 노출되도록 처리

결과물

33.mp4

리뷰어에게 추가로 요구하는 사항 (선택)

  • 상세 화면이 스크롤 컨테이너가 아니어서 직접 세로 드래그를 감지하는 방식으로 구현했는데 trigger/hold 값이 사용감상 적절한지 확인 부탁해 !

@Keepiluv Keepiluv deleted a comment from coderabbitai Bot Jun 7, 2026
@chanho0908 chanho0908 changed the title Feat: 인증샷 상세 당겨서 새로고침 추가 인증샷 상세 당겨서 새로고침 추가 Jun 7, 2026
@chanho0908 chanho0908 self-assigned this Jun 7, 2026
@chanho0908 chanho0908 added the Feature Extra attention is needed label Jun 7, 2026
@chanho0908 chanho0908 requested a review from dogmania June 7, 2026 01:16
@coderabbitai

coderabbitai Bot commented Jun 7, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@chanho0908, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 30 minutes and 5 seconds. Learn how PR review limits work.

Your organization has run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro Plus

Run ID: 2a720d3f-4591-49c9-bcc6-aab5e445f81c

📥 Commits

Reviewing files that changed from the base of the PR and between dbbd63a and aded2c5.

📒 Files selected for processing (5)
  • feature/photolog/detail/src/main/java/com/twix/photolog/detail/PhotologDetailScreen.kt
  • feature/photolog/detail/src/main/java/com/twix/photolog/detail/PhotologDetailViewModel.kt
  • feature/photolog/detail/src/main/java/com/twix/photolog/detail/component/reaction/ReactionContent.kt
  • feature/photolog/detail/src/main/java/com/twix/photolog/detail/contract/PhotologDetailIntent.kt
  • feature/photolog/detail/src/main/java/com/twix/photolog/detail/contract/PhotologDetailUiState.kt
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/#169-photolog-detail-refresh

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@dogmania dogmania left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

제가 테스트해보니까 이게 SwipeableCard 영역을 벗어나서 상하 드래그를 해야 리프레시가 되더라구요 이 영역이 조금 좁은 감이 있어서 수정하면 좋을 것 같아요.
제가 간단하게 수정해 봤을 때는 SwipeableCard에 있는 detectDragGestures를 detectHorizontalDragGestures로 변경해서 카드는 좌우 이벤트만 소비하고 상하 이벤트는 소비하지 않게 수정하면 카드 위에서 상하 드래그를 해도 리프레시 가능한 것 같습니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature Extra attention is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants