Skip to content

Comments

인증샷 업로드시 로딩 화면 추가#100

Open
chanho0908 wants to merge 6 commits intodevelopfrom
feat/#99-photolog-loading
Open

인증샷 업로드시 로딩 화면 추가#100
chanho0908 wants to merge 6 commits intodevelopfrom
feat/#99-photolog-loading

Conversation

@chanho0908
Copy link
Member

이슈 번호

#99

작업내용

  • 인증샷 로딩 화면 구현

결과물

default.mp4

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

@chanho0908 chanho0908 self-assigned this Feb 24, 2026
@chanho0908 chanho0908 linked an issue Feb 24, 2026 that may be closed by this pull request
1 task
@chanho0908 chanho0908 added the Feature Extra attention is needed label Feb 24, 2026
coderabbitai[bot]

This comment was marked as resolved.

@chanho0908 chanho0908 requested a review from dogmania February 24, 2026 11:05
Copy link
Member

@dogmania dogmania 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
Member

Choose a reason for hiding this comment

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

BaseViewModel 보면 onStart, onFinally가 로딩 상태 처리하라고 만든 거라서 혹시 지금 상황에서 사용하지 않은 이유가 따로 있나요??

Copy link
Member Author

Choose a reason for hiding this comment

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

몰랐슴둥 ... 🙏
리뷰 반영 커밋 : 202ba71

import com.twix.domain.model.enums.AppTextStyle

@Composable
internal fun LoadingContent() {
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.

좋은 생각인 것 같아 !

리뷰 반영 커밋 : aca38cf

}

@Composable
private fun FloatingPlane() {
Copy link
Member

Choose a reason for hiding this comment

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

6f, 8f가 반복적으로 사용되니까 변수로 빼는 게 나중에 수정할 때 좋을 거 같아요!

Copy link
Member Author

Choose a reason for hiding this comment

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

리뷰 반영 커밋 : 2fc56a3

- `offsetY`, `offsetX`, `rotation` 애니메이션에 사용되는 매직 넘버를 상수로 추출하여 가독성 및 유지보수성 향상
@YAPP-Github YAPP-Github deleted a comment from coderabbitai bot Feb 25, 2026
coderabbitai[bot]

This comment was marked as resolved.

@YAPP-Github YAPP-Github deleted a comment from coderabbitai bot Feb 25, 2026
@chanho0908 chanho0908 requested a review from dogmania February 25, 2026 09:40
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