Skip to content

[Feat] 운동 기록 조회 리팩토링 및 스켈레톤 구현 #386#393

Merged
edv-Shin merged 13 commits intodevfrom
feat/loading-386
Dec 1, 2025
Merged

[Feat] 운동 기록 조회 리팩토링 및 스켈레톤 구현 #386#393
edv-Shin merged 13 commits intodevfrom
feat/loading-386

Conversation

@edv-Shin
Copy link
Copy Markdown
Contributor

@edv-Shin edv-Shin commented Nov 18, 2025

#️⃣연관된 이슈

ex) #이슈번호, #이슈번호

#386

📝작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요(이미지 첨부 가능)

viewModel에서 savedStateHandle로 가져오던 recordId를 navArgs로 가져오도록 수정했습니다. 0f061d4 de1ae98

onResume에서 조회 api를 호출하던 로직을 리팩토링 했습니다.

  • 갱신이 필요할 때만 api를 호출하도록 수정했습니다.
  • setupViews에서 getExerciseRecord를 호출하기 때문에 조회 로딩시간이 단축됐습니다

운동 기록 조회 화면에 스켈레톤 로딩을 추가했습니다. 6e16395

  • 공통으로 사용할 수 있는 UiState를 만들어 Loading, Error, Success 상태를 표시할 수 있도록 구현했습니다.
  • Failure 타입에 따라 에러 문구를 표시할 수 있도록 구현했습니다.

스크린샷 (선택)

2025-11-18.18.36.38.mov

💬리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

조회 시에 로딩시간이 매우 짧으면 스켈레톤 ui가 오히려 그냥 깜빡이는 것처럼 보이기도 합니다.
그래서 스켈레톤 ui를 일관적으로 보여줄 수 있도록 최소 로딩 시간으로 delay(300ms)를 넣었는데, 어떤게 좋은 방법인지 고민입니다.

  1. 근본적인 고민인데, 스켈레톤이 깜빡임처럼 보이더라도 빠르게 표시해주는게 좋을까요?
  2. 스켈레톤을 최소 로딩시간동안으로 보여준다고 가정했을 때, 지금 로직으로는 총 로딩 시간이 (delay + api 호출대기시간)이 됩니다. 로딩 시간을 api 호출 대기 시간과 delay 중 더 큰 값으로 나타내고 싶은데, 구현할 방법이 어떤게 있을까요?

@edv-Shin edv-Shin requested a review from yunjaena November 18, 2025 09:35
@edv-Shin edv-Shin self-assigned this Nov 18, 2025
@edv-Shin edv-Shin linked an issue Nov 18, 2025 that may be closed by this pull request
1 task
* 키보드가 내려갈 때는 네비게이션 바 높이만큼 패딩 조정
*/
private fun setupKeyboardAdjustments() {
ViewCompat.setOnApplyWindowInsetsListener(binding.scrollView) { v, insets ->
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.

EdgeToEdge가 활성화 되어있나요?

Copy link
Copy Markdown
Contributor Author

@edv-Shin edv-Shin Nov 20, 2025

Choose a reason for hiding this comment

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

MainActivity에 활성화 설정을 추가했습니다.

+키보드에 맞게 패딩을 추가하는 로직이 프로필 편집화면에도 필요한 것을 확인했습니다.
하단 패딩 조절 로직을 KeyboardAdjustHelper로 공통화해서 사용할 수 있도록 수정했습니다.
8157524
6fb0adb

@yunjaena
Copy link
Copy Markdown
Contributor

Systembars에서는 어떤 것들이 포함되어있나요?

@edv-Shin
Copy link
Copy Markdown
Contributor Author

Systembars에서는 어떤 것들이 포함되어있나요?

StatusBar랑 Navigation Bar가 포함되어 있는걸로 알고있습니다.

# Conflicts:
#	feature/chatting/src/main/java/com/project200/feature/chatting/chattingRoom/ChattingRoomFragment.kt
@edv-Shin edv-Shin merged commit 5a6e5b6 into dev Dec 1, 2025
1 check passed
@edv-Shin edv-Shin deleted the feat/loading-386 branch January 29, 2026 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feat] 로딩 스켈레톤 구현

2 participants