Skip to content

Comments

design: 중간 피드백 반영 Auth페이지 UI 수정 및 추가#111

Merged
Seony777 merged 19 commits intodevelopfrom
design/93-auth-css
Feb 2, 2026
Merged

design: 중간 피드백 반영 Auth페이지 UI 수정 및 추가#111
Seony777 merged 19 commits intodevelopfrom
design/93-auth-css

Conversation

@Seony777
Copy link
Collaborator

@Seony777 Seony777 commented Feb 1, 2026

📌 관련 이슈번호

close : #93

🔍 구현한 내용

  • 로그인/회원가입 페이지의 gap을 수정하였습니다.
  • 회원가입 페이지에서 중복확인 성공 시에 버튼 disabled로 막았습니다.
  • 아이디/비밀번호 찾기 페이지에서 [아이디 찾기 | 비밀번호 찾기] 버튼, 소셜 로그인 버튼, 회원가입하기 버튼을 삭제하고 정렬을 맞췄습니다.
  • 온보딩 조합 생성하기 페이지의 입력창 크기와 gap을 수정했습니다.
  • 온보딩의 맨 마지막에 기기 추천 페이지의 UI를 생성하고 라우팅 세팅을 하였습니다.
  • 아이디/비밀번호 찾기, 회원가입 하기 등의 회색 텍스트에 hover를 추가하였습니다.

📸 스크린샷 or 실행 영상

image

📢 리뷰어에게

  • 기기 추천 페이지의 api연동은 추후에 진행하겠습니다.

@Seony777 Seony777 requested review from H-un1 and waldls February 1, 2026 12:13
@Seony777 Seony777 added 🎨 UI 마크업 & 스타일링 🍀 선우 labels Feb 1, 2026
@github-actions
Copy link

github-actions bot commented Feb 1, 2026

@github-actions
Copy link

github-actions bot commented Feb 1, 2026

@H-un1
Copy link
Collaborator

H-un1 commented Feb 2, 2026

  1. SignupAccountPage.tsx
    현재 이메일 입력창을 다 지웠을 때만 false가 되는 것으로 보입니다. 사용자가 comm과 같이 입력하여도 true로 될 수 있습니다. onChange에서 값을 변경할 때마다 검증 상태를 초기화해야 합니다. register의 onChange에서 처리하기에 useEffect는 불필요할 수 있습니다. register 내부의 onChange가 잘 작동한다면 useEffect는 삭제해도 됩니다.

  2. OnboardingCombinationPage.tsx
    PrimaryButton이 disabled 처리되어 있지만, 함수 내부에서도 isPending 체크를 한 번 더 하면 제출 중복 방지가 될 것 같습니다.

alert('조합 생성에 실패했습니다.') 도 유틸 함수 사용하여 구체적인 에러 메세지 보여주는 것이 좋을 것 같습니다. (ex. 이미 존재하는 조합명입니다.)

  1. OnboardingRecommendationPage.tsx
    새로고침을 하면 queryClient의 캐시가 날아갈 수 있습니다.

  2. FindPasswordStep/Step1Form.tsx
    errors.email이 존재하면 errors.email은 truthy여서 옆의 조건은 무조건 참입니다. 연산자가 필요 없을 것 같습니다.

@Seony777
Copy link
Collaborator Author

Seony777 commented Feb 2, 2026

전부 수정했습니다.
2-2는 온보딩 과정에서의 조합 생성하기라 네트워크 오류 외에는 나타날 오류가 없어 유지했습니다.

@github-actions
Copy link

github-actions bot commented Feb 2, 2026

Copy link
Member

@waldls waldls left a comment

Choose a reason for hiding this comment

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

확인했습니다 수고하셨어요!

@github-actions
Copy link

github-actions bot commented Feb 2, 2026

@Seony777 Seony777 merged commit b7bbcb5 into develop Feb 2, 2026
1 check passed
@Seony777 Seony777 deleted the design/93-auth-css branch February 2, 2026 15:39
waldls added a commit to waldls/DeviceLife_Frontend that referenced this pull request Feb 2, 2026
* design: secondaryButton disabled버전 추가

* design: 로그인/회원가입 화면들 gap수정

* design: 회색 안내 문구 hover추가

* design: 온보딩 css수정

* feat: 기기조합 추천 페이지 레이아웃 구현 및 라우팅 연결

* design: 회원가입 버튼 및 입력창 width/height 수정

* feat: 중복확인 성공시 버튼 비활성화

* feat: 온보딩 기기조합 추천 페이지 navigate연결

* refactor: 온보딩 페이지 흐름 수정

* feat: 기기 추천 페이지 텍스트 및 간격 수정

* design: 아이디/비밀번호 찾기 페이지,아이디 찾기 | 비밀번호 찾기 문구, 소셜로그인 버튼, 회원가입 하기 문구 제거

* design: 다음에 하기 버튼 구현

* feat: 다음에 하기 버튼 클릭 시 홈으로 이동

* refactor: 중복 로직 제거

* fix: 새로고침 시에도 유저 프로필 정보 유지

* refactor: 중복 조건 제거
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🍀 선우 🎨 UI 마크업 & 스타일링

Projects

None yet

Development

Successfully merging this pull request may close these issues.

design: 중간 피드백 반영 회원가입/로그인/온보딩 디자인 수정 및 추가

3 participants