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

[ Feature/workbook 41 ] Skeleton 화면 구현 #44

Merged
merged 16 commits into from
Jun 15, 2024

Conversation

soomin9106
Copy link
Collaborator

🔥 Related Issues

resolve #41
close #41

💜 작업 내용

  • workbook msw 핸들러 delay 구현
  • Skeleton 컴포넌트 구현

✅ PR Point

  • 딜레이 util 추가
    `export const _1_SECOND = 1_000;
    export const _3_SECOND = 3_000;

export const delay = (second = _3_SECOND) => new Promise((resolve) => setTimeout(resolve, second));`

  • Shadcn UI Skeleton 컴포넌트 이용하여 워크북 진입 스켈레톤 UI 구현

👀 스크린샷 / GIF / 링크

스크린샷 2024-06-14 오후 12 29 50

Copy link

cloudflare-workers-and-pages bot commented Jun 14, 2024

Deploying 24th-web-team-1-fe with  Cloudflare Pages  Cloudflare Pages

Latest commit: bd4a8ab
Status: ✅  Deploy successful!
Preview URL: https://2bcd2dea.24th-web-team-1-fe.pages.dev
Branch Preview URL: https://feature-workbook-41.24th-web-team-1-fe.pages.dev

View logs

@soomin9106 soomin9106 requested a review from Happhee June 14, 2024 07:30
<Skeleton className="h-[30px] w-full" style={{ marginTop: "9px" }} />
<Skeleton className="h-[30px] w-full" style={{ marginTop: "9px" }} />
<Skeleton className="h-[30px] w-full" style={{ marginTop: "9px" }} />
<Skeleton className="h-[30px] w-full" style={{ marginTop: "9px" }} />
Copy link
Collaborator

Choose a reason for hiding this comment

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

이거 contentSkelton으로 분리하고 map 돌리면 어떨까요

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

좋아요 !! 다른 데에서도 쓰일 법하니 common 으로 빼둘게요!!
길이 & 스타일은 props 로 넣을 수 있도록 할게용 !!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Copy link
Collaborator

@Happhee Happhee left a comment

Choose a reason for hiding this comment

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

빠른작업속도 굿굿!

@soomin9106 soomin9106 merged commit a9247ce into develop Jun 15, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants