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

feat(fe): add skeleton UI #50

Merged
merged 2 commits into from
Jan 24, 2025
Merged

feat(fe): add skeleton UI #50

merged 2 commits into from
Jan 24, 2025

Conversation

cjeongmin
Copy link
Contributor

@cjeongmin cjeongmin commented Jan 23, 2025

개요

데이터 로딩 지연 시 사용자 경험을 개선하기 위해 스켈레톤 UI를 구현했습니다. 적용 컴포넌트는 다음과 같습니다.

  • ChattingList
  • QuestionList
  • ReplyList

네트워크 지연시에 자연스러운 로딩 화면을 제공하여 사용자의 체감 대기 시간을 최소화했습니다.

이슈

@cjeongmin cjeongmin self-assigned this Jan 23, 2025
@Copilot Copilot bot review requested due to automatic review settings January 23, 2025 13:04

Choose a reason for hiding this comment

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

Copilot reviewed 5 out of 9 changed files in this pull request and generated no comments.

Files not reviewed (4)
  • apps/client/src/widgets/question-list/index.ts: Evaluated as low risk
  • apps/client/src/widgets/question-list/ui/QuestionListSkeletonUI.tsx: Evaluated as low risk
  • apps/client/src/widgets/reply-list/index.ts: Evaluated as low risk
  • apps/client/src/widgets/chatting-list/index.ts: Evaluated as low risk
Comments suppressed due to low confidence (1)

apps/client/src/pages/session/ui/QuestionReplyPage.tsx:11

  • Ensure that the new skeleton UI components (ReplyListSkeletonUI, ChattingListSkeletonUI, QuestionListSkeletonUI) are covered by tests.
  <Suspense fallback={<ReplyListSkeletonUI />}>
@wlgh1553 wlgh1553 self-requested a review January 23, 2025 13:37
Copy link
Contributor

@wlgh1553 wlgh1553 left a comment

Choose a reason for hiding this comment

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

사용자 경험의 신 최마군요
고생이 많으십니다 LGTM

@shl0501 shl0501 self-requested a review January 23, 2025 14:25
Copy link
Collaborator

@shl0501 shl0501 left a comment

Choose a reason for hiding this comment

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

최근 사용자의 로딩 체감 시간을 어떻게 처리할 수 있는지에 대한 고민이 많았었는데 덕분에 좋은 예시를 보고 갑니다.
LGTM

@cjeongmin cjeongmin merged commit 2f0522a into main Jan 24, 2025
2 checks passed
@cjeongmin cjeongmin deleted the feat(fe)/skeleton-ui branch January 24, 2025 00:23
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.

[Feat]: 스켈레톤 UI 추가
3 participants