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

[#3] step1 페이지 ui 및 페이지간 이동 구현 #8

Merged
merged 14 commits into from
Apr 6, 2024
Merged

Conversation

03hoho03
Copy link
Collaborator

@03hoho03 03hoho03 commented Apr 6, 2024

작업 내용

image
전체적인 ui 를 작성했습니다.

다음 단계 버튼을 누르면 query string이 step=2 로 변하며 step2 로 이동하게 됩니다.
textarea 는 최대 500자 까지 적을 수 있습니다.
상단에서 진행 상황을 볼 수 있으며 클릭시 해당 step 으로 바로 이동할 수 있습니다.
상단 좌측의 left chevron 버튼을 누르면 경고 문구가 alert 됩니다. 예를 누르게 된다면 메인 페이지로 빠져나가게 됩니다.

중요한 포인트

!! textarea 의 데이터는 현재 저장되지 않습니다. 추후 구현 예정입니다. (컴포넌트 unmount 시 저장 x)

@03hoho03 03hoho03 merged commit fb88de7 into develop Apr 6, 2024
2 checks passed
@03hoho03 03hoho03 deleted the feat/#3 branch April 6, 2024 10:42
Comment on lines +7 to +22
function PaginationBtn({ step }: { step: number }) {
const params = useSearchParams()

const stepParam = params.get('step')

return (
<Link
className={cn(S.btn, {
[S.darken]: stepParam && parseInt(stepParam) === step,
})}
href={`/verification/ibulsin?step=${step}`}
>
{step}
</Link>
)
}
Copy link

Choose a reason for hiding this comment

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

useSearchParams Suspense를 감싸서 해결하는게 아니라 use client를 이용해서 client componetn로 써야될거같네요

Comment on lines +8 to +10
{Array.from({ length: 3 }, (_, idx) => (
<PaginationBtn key={idx} step={idx + 1} />
))}
Copy link

Choose a reason for hiding this comment

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

그냥 이럴떄는 [1,2,3].map() 형식으로 많이 씁니다

Comment on lines +14 to +20
<body>
<Providers>
<Recoil>
<Layout>{children}</Layout>
</Recoil>
</Providers>
</body>
Copy link

Choose a reason for hiding this comment

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

👍 useRouter issue

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.

2 participants