Skip to content

fix(sign-up): 이메일 회원가입 하단 버튼 정렬 수정#597

Merged
wibaek merged 1 commit into
mainfrom
fix/email-signup-button-alignment
Jul 5, 2026
Merged

fix(sign-up): 이메일 회원가입 하단 버튼 정렬 수정#597
wibaek merged 1 commit into
mainfrom
fix/email-signup-button-alignment

Conversation

@wibaek

@wibaek wibaek commented Jul 4, 2026

Copy link
Copy Markdown
Member

요약

  • 모바일 이메일 회원가입 화면에서 하단 버튼이 오른쪽으로 치우쳐 보이는 문제를 수정했습니다.
  • 하단 fixed 영역을 viewport 기준으로 고정하고 내부 컨테이너에서 앱 폭과 좌우 여백을 맞췄습니다.

변경 사항

  • 이메일 회원가입 하단 버튼 wrapper에 left-0 right-0을 적용했습니다.
  • 내부 버튼 컨테이너에 mx-auto w-full max-w-app px-5를 적용해 기존 앱 레이아웃과 같은 좌우 여백을 유지했습니다.

검증

  • pnpm --filter @solid-connect/web run lint
  • pnpm --filter @solid-connect/web run typecheck
  • pre-push @solid-connect/web ci:check 및 build 통과
  • 모바일 390px/360px viewport에서 버튼 좌우 여백 확인

노트

  • 없음

@wibaek wibaek requested review from enunsnv and manNomi as code owners July 4, 2026 17:16
@vercel

vercel Bot commented Jul 4, 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 4, 2026 5:19pm
solid-connection-web Ready Ready Preview, Comment Jul 4, 2026 5:19pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
solid-connect-web-admin Skipped Skipped Jul 4, 2026 5:19pm

@github-actions github-actions Bot added the web label Jul 4, 2026
@coderabbitai

coderabbitai Bot commented Jul 4, 2026

Copy link
Copy Markdown

Review Change Stack

Walkthrough

이번 PR은 회원가입 이메일 화면(EmailSignUpForm)의 하단 고정 버튼 영역 스타일만 살짝 손질했어요! 기존 fixed 컨테이너에 left-0 right-0 w-full을 추가하고, 내부 래퍼에는 mx-auto, w-full, max-w-app을 넣어서 폭과 정렬 방식을 다듬었답니다. 로직 변경은 전혀 없고, 순수하게 마크업/클래스 구성만 손봤어요.

Changes

  1. 하단 고정 버튼 영역 스타일 조정 (apps/web/src/app/sign-up/email/EmailSignUpForm.tsx)
    • fixed 컨테이너에 left-0 right-0 w-full 클래스 추가
    • 내부 래퍼 클래스를 mb-[37px] px-5에서 mx-auto mb-[37px] w-full max-w-app px-5로 변경

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

Suggested reviewers: enunsnv, khwww

시(詩) 한 수 🐰

토끼 한 마리, 코드를 살짝 다듬고,
왼쪽 오른쪽 여백을 가지런히 심었죠.
가운데 정렬 마법, mx-auto 한 스푼,
버튼은 이제 화면 속에 딱 맞게 쏙!
깡총깡총, 리뷰도 가볍게 끝~ 🥕

🚥 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/email-signup-button-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.

@wibaek wibaek self-assigned this Jul 4, 2026
@wibaek wibaek merged commit 722afa5 into main Jul 5, 2026
13 checks passed
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