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.

Sign up for GitHub

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

Jump to bottom

Issue 45 feat 로그인UI 퍼블리싱 #37

Merged
merged 18 commits into from
Oct 28, 2024
Merged

Issue 45 feat 로그인UI 퍼블리싱 #37

merged 18 commits into from
Oct 28, 2024

Conversation

CosmicSandBox
Copy link
Contributor

@CosmicSandBox CosmicSandBox commented Oct 23, 2024 •
edited
Loading

📌 관련 이슈

✨ PR 세부 내용

로그인 UI,
회원가입 UI
퍼블리싱 완료 했습니다!

추가적으로 로그인, 회원가입에 들어가는 Input Component도 2가지 따로 개발하였습니다.
(LoginInput.jsx, SignUpInput.jsx)

⌛ 소요 시간

8시간

@CosmicSandBox
[ISSUE-45] feat: 로그인 기본 틀 제작 (#31)
31a9be1
@CosmicSandBox
[ISSUE-45] feat: 아이디, 비밀번호 Input 컴포넌트 생성 (#31)
fab2e56
@CosmicSandBox
[ISSUE-45] feat: 눈에 보이는 모든 UI 추가 완료. 다음 커밋부터 세세한 UI 조정시작 (#31)
2ad5217
@CosmicSandBox
[ISSUE-45] UI: login 화면 input icon, radio button, ?버튼 제외 UI 완성 (#31)
0b3e6c2
@CosmicSandBox
[ISSUE-45] feat: sign-up 페이지 제작 시작 (#31)
6f6fd9d
@CosmicSandBox
[ISSUE-45] move: AuthInput을 LoginInput 이름으로 바꾸고 Login 폴더내 component 폴…
aabcc58 
…더로 이동 (#31)
@CosmicSandBox
[ISSUE-45] feat: 회원가입 페이지 input field 제외 UI 추가 완료 (#31)
feec618
@CosmicSandBox
[ISSUE-45] feat: 회원가입 페이지 라우팅 추가 및 input field ui antdesign 사용 (#31)
e563ff2
@CosmicSandBox
[ISSUE-45] UI: 로그인 페이지 antdesign 사용 (#31)
927b598
@CosmicSandBox
[ISSUE-45] UI: 비밀번호 Show 부분 underline 추가 완료 (#31)
2f67323
@CosmicSandBox
[ISSUE-45] UI: input field 폰트 크기 수정 및 자동 로그인 PRETENDARD 적용 (#31)
f1c91cf
@CosmicSandBox
[ISSUE-45] UI: radio button ant design 적용 및 회원가입 페이지 유효성 오류 메세지 위치 수정 (…
af84d87 
…#31)
@CosmicSandBox
[ISSUE-45] UI: 로그인 우측 상단 ? 버튼 추가 (#31)
b69377e
@CosmicSandBox
[ISSUE-45] UI: 로그인 우측 상단 ? 버튼 위치 고정 (#31)
18440ce
@CosmicSandBox
[ISSUE-45] UI: 간격 조정 및 여백 문제 해결 (#31)
3ac5918
InKyungWoo
InKyungWoo reviewed Oct 25, 2024
View reviewed changes
src/app/(auth)/login/page.jsx Outdated Show resolved Hide resolved
src/app/(auth)/sign-up/components/SignUpInput.jsx Outdated Show resolved Hide resolved
src/assets/images/login-question.png Outdated Show resolved Hide resolved
InKyungWoo
InKyungWoo requested changes Oct 25, 2024
View reviewed changes
src/app/(auth)/login/page.jsx Outdated Show resolved Hide resolved
src/app/(auth)/login/page.jsx Show resolved Hide resolved
InKyungWoo
InKyungWoo requested changes Oct 25, 2024
View reviewed changes
src/app/(auth)/login/page.jsx Outdated Show resolved Hide resolved
@CosmicSandBox
Copy link
Contributor Author

말씀해주신 부분들 수정 하였고
회의때 보였던 문제점 : SignUp 페이지에서 Show가 두개 묶여서 동작하는 것 -> 개별적으로 동작하도록 수정하였고
login 페이지에 padding bottom을 실수로 주어서 해당 값을 삭제하였습니다!
좋은 코드 리뷰 감사합니다! 덕분에 수정하면서 또 많이 배웠습니다!!

@InKyungWoo
Copy link
Contributor

InKyungWoo commented Oct 28, 2024 •
edited
Loading

👨‍💻 10/28 페어프로그래밍 진행 내용:

  • 뷰포트 스크롤 문제 해결
  • next/Image 컴포넌트로 수정
  • 코드 리팩토링
  • Antd 체크박스 컴포넌트 css(hover) 오버라이딩
  • 네이버 아이콘 추가

@InKyungWoo InKyungWoo merged commit cf6837b into dev Oct 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@InKyungWoo InKyungWoo InKyungWoo approved these changes

Assignees

@CosmicSandBox CosmicSandBox

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Feat: 로그인, 회원가입 UI 퍼블리싱
2 participants
@CosmicSandBox @InKyungWoo