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

Base 김주동 sprint2 #29

Conversation

joodongkim
Copy link
Collaborator

요구사항

기본

  • 아래로 스크롤 해도 상단 네비게이션바 고정됩니다.
  • “판다마켓" 클릭 시 루트 페이지(“/”)로 이동합니다.
  • [x]로그인 페이지 구현
  • [x]“로그인”버튼 클릭 시 (“/login”) 페이지로 이동합니다.
  • [x]회원가입 페이지 구현
  • [x]“회원 가입” 버튼 클릭 시 “/joinmember” 페이지로 이동합니다.
  • [x]input 요소에 focus in / out 테두리 색상 변경합니다.
  • [x]구글/카카오 아이콘 클릭시 각각늬 페이지로 이동합니다.

심화

  • [x]비밀번호 input 요소 오른쪽에 눈모양 아이콘을 추가합니다.

주요 변경사항

스크린샷

image

멘토에게

  • 셀프 코드 리뷰를 통해 질문 이어가겠습니다.

@joodongkim joodongkim closed this Aug 17, 2024
@joodongkim joodongkim reopened this Aug 19, 2024
@GANGYIKIM GANGYIKIM changed the base branch from main to Basic-김주동 August 19, 2024 01:46
Copy link
Collaborator

@GANGYIKIM GANGYIKIM left a comment

Choose a reason for hiding this comment

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

주동님 두번째 스프린트 미션 고생하셨습니다.
결과물도 코드도 너무 잘 하셨습니다!
덕분에 크게 첨언할 것이 없었습니다.
다음주 미션도 화이팅입니다!

+다음에는 base branch를 main이 아니라 Basic-김주동으로 설정해주세요.


  • 동일한 내용에 대해서는 코드에서 한번만 코멘트를 답니다.
  • 이미지 명 비슷하게 네이밍 해주신 거 좋습니다.
  • commit 단위를 조금 더 쪼개서 어떤 작업을 하셨는지 알 수 있게 하면 좋겠습니다.

<br />쉽게 찾아보세요
</p>
</div>
<img src="img/feature2-image.png" alt="검색 기능" width="50%" />
Copy link
Collaborator

Choose a reason for hiding this comment

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

P3:
나중에 반응형 처리하시려면 css 로 이미지 사이즈 컨트롤하시는게 더 좋습니다.

</div>
</div>
<div id="features" class="wrapper">
<ul style="list-style-type: none">
Copy link
Collaborator

Choose a reason for hiding this comment

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

P2:
인라인 스타일링보다 css파일에 작성하시는거 추천드립니다.

Suggested change
<ul style="list-style-type: none">
<ul>

<meta charset="UTF-8" />
<meta name="viewport" content="width=device-width, initial-scale=1.0" />
<title>판다마켓 - 회원가입</title>
<link rel="icon" href="img/favicon.ico" />
Copy link
Collaborator

Choose a reason for hiding this comment

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

P3:
파비콘 설정해주신거 좋습니다 👍

Comment on lines +27 to +28
<label for="email">이메일</label>
<input
Copy link
Collaborator

Choose a reason for hiding this comment

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

P3:
라벨과 인풋 연결해주신거 좋습니다 👍

Comment on lines +53 to +57
<img
src="img/hide_input.svg"
alt="비밀번호 숨김"
class="toggle-password"
/>
Copy link
Collaborator

Choose a reason for hiding this comment

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

P3:
나중에 버튼으로 작동할거라 button 태그로 작성해주시면 좋겠습니다.

Comment on lines +77 to +80
<button type="submit" class="button pill-button full-width">
회원가입
</button>
</form>
Copy link
Collaborator

Choose a reason for hiding this comment

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

P2:
submit type button과 form 태그 같이 써주신 거 좋습니다 👍

@GANGYIKIM GANGYIKIM merged commit cdf622c into codeit-bootcamp-frontend:Basic-김주동 Aug 20, 2024
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