Skip to content

design(signup): 가입 준비 단계 정렬 조정#601

Open
wibaek wants to merge 1 commit into
mainfrom
fix/signup-prepare-alignment
Open

design(signup): 가입 준비 단계 정렬 조정#601
wibaek wants to merge 1 commit into
mainfrom
fix/signup-prepare-alignment

Conversation

@wibaek

@wibaek wibaek commented Jul 7, 2026

Copy link
Copy Markdown
Member

요약

  • 회원가입 준비 단계 화면에서 바깥 padding 중복으로 카드 폭 기준이 어긋나 보이던 부분을 정리했습니다.
  • 준비 단계 카드의 아이콘 영역을 고정 폭 슬롯으로 맞춰 각 카드의 텍스트 시작점이 동일하게 보이도록 조정했습니다.
  • 버튼 focus outline을 primary 색상으로 명시해 선택/포커스 상태가 화면 톤과 맞게 보이도록 했습니다.

변경 사항

  • sign-up 페이지 wrapper의 중복 px-5 제거
  • 준비 단계 선택 카드의 아이콘 슬롯을 w-16으로 고정하고 텍스트 영역을 min-w-0로 안정화
  • 준비 단계 선택 카드의 focus-visible outline 색상을 primary로 지정

검증

  • pnpm --filter @solid-connect/web run ci:check
  • push hook에서 pnpm run lint:check && pnpm run typecheck:ci, next build 통과
  • 로컬 브라우저에서 /sign-up?token=test 진입 후 준비 단계 카드 정렬 확인

노트

@wibaek wibaek requested a review from manNomi as a code owner July 7, 2026 17:07
@github-actions github-actions Bot added the web label Jul 7, 2026
@vercel

vercel Bot commented Jul 7, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
solid-connect-university-web Ready Ready Preview, Comment Jul 7, 2026 5:08pm
solid-connection-web Ready Ready Preview, Comment Jul 7, 2026 5:08pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
solid-connect-web-admin Skipped Skipped Jul 7, 2026 5:08pm

@vercel vercel Bot temporarily deployed to Preview – solid-connect-web-admin July 7, 2026 17:07 Inactive
@coderabbitai

coderabbitai Bot commented Jul 7, 2026

Copy link
Copy Markdown

Review Change Stack

Walkthrough

이번 PR은 두 가지 스타일 변경 사항을 담고 있습니다.

Changes

  1. 회원가입 페이지 패딩 제거
    • 최상단 레이아웃 컨테이너의 className에서 px-5를 삭제, 좌우 패딩이 사라짐
  2. PrepareChoiceButton UI 재구성
    • 버튼 className을 공통 Tailwind 문자열 기반으로 단순화
    • 선택 상태에 따른 스타일을 border-primary bg-primary-100 / border-white bg-white로 정리
    • 아이콘 영역을 패딩 기반에서 고정 폭/정렬(flex w-16 shrink-0 justify-center) 기반으로 전환
    • 텍스트 컨테이너에 min-w-0 추가

Estimated code review effort: 1 (Trivial) | ~3 minutes

Suggested reviewers: enunsnv, khwww

Poem

  • 패딩 하나 쏙 빼내니, 화면이 한결 넓어졌네 🐰
  • 버튼 옷도 새로 입혀, 선택할 땐 색이 빛나네 ✨
  • 작은 클래스 몇 줄이지만, 정성 담아 다듬었죠
  • 토끼는 오늘도 통통, 코드밭을 깡충깡충 🥕
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed 제목이 회원가입 준비 단계의 정렬 조정이라는 주요 변경을 잘 요약합니다.
Description check ✅ Passed 요약, 변경 사항, 검증이 포함되어 핵심 내용은 충분하며, 일부 선택 섹션만 비어 있습니다.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/signup-prepare-alignment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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.

1 participant