Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
잘 하셨어요! 저번 멘토링때 말씀드린것 처럼 정답은 없답니다 ㅎㅎ 다만 이제 개인적인 취향에 따라 좀 갈리는 부분이 있긴 한데요. |
1005hoon
reviewed
Nov 6, 2024
| alt="판다마켓 로고" | ||
| /> | ||
| <span class="header__logo-text">판다마켓</span> | ||
| </a> | ||
| <a href="/login" class="header__login-button">로그인</a> | ||
| <a href="/login.html" class="header__login-button">로그인</a> |
There was a problem hiding this comment.
/login 으로 했을경우 사진과 같이 나와서 .html을 붙여줬습니다. 혹여 문제가 될까요 ?
| <a href="/" class="logo__link"><h1>판다마켓</h1></a> | ||
| </div> | ||
| </header> |
| <p>간편 로그인하기</p> | ||
| <div class="sns-login__icon"> | ||
| <div class="social-login__icon"> |
|
@CJewon 작업 다 마치시구, 우측 상단에 리뷰요청하기 다시 눌러주시면 확인해볼게요! |
|
코드 수정은 /login과 /login.html 오류 부분을 확인해주시면 merge 받고, sprint3 미션과 함께 작업해서 sprint3 미션을 마친 후, 확인받는 PR을 요청드릴때 수정사항으로 작성해놓겠습니다 ! 감사합니다 ! |
CJewon
added a commit
to CJewon/12-Sprint-Mission
that referenced
this pull request
Nov 8, 2024
* div태그 대신 header 태그로 변경 및 logo-img에 대한 a태그 추가 * 이미지 파일구조 수정 및 파일구조 수정에 따른 이미지들의 src 수정 * login.html form태그와 label태그로 변경 및 비밀번호 가림 이미지 추가와 그에 맞는 css 적용 * signup.html form태그와 label태그로 변경 및 비밀번호 가림 이미지 추가와 그에 맞는 css 적용
kich555
pushed a commit
that referenced
this pull request
Nov 17, 2024
* reset * fix: 머지 후 브랜치 삭제 github action 수정 * env: workflows 폴더로 이동 * codeit sprint mission part1 update (#13) * BEM 네이밍 및 시멘틱 구조 개선 * BEM에 맞게 HTML의 class 명칭과 CSS 명칭 수정 * codeit-sprint2 mission 로그인 홈페이지와 회원가입 홈페이지 제작 (#68) * [최제원] Sprint2 (#102) * div태그 대신 header 태그로 변경 및 logo-img에 대한 a태그 추가 * 이미지 파일구조 수정 및 파일구조 수정에 따른 이미지들의 src 수정 * login.html form태그와 label태그로 변경 및 비밀번호 가림 이미지 추가와 그에 맞는 css 적용 * signup.html form태그와 label태그로 변경 및 비밀번호 가림 이미지 추가와 그에 맞는 css 적용 * [최제원] Sprint3 (#114) * 메인 홈페이지 반응형 웹으로 변경함으로써 CSS 수정 및 변경 * login 홈페이지와 signup 홈페이지를 반응형 웹으로 변경함으로써 CSS 수정 및 변경 * 불필요한 CSS 삭제 및 BEM에 맞게 클래스 이름 수정 * 로그인 홈페이지와 회원가입 홈페이지의 에러메세지 관련 로직 구현 * input에 aria-label 속성 추가 --------- Co-authored-by: hanseulhee <3021062@gmail.com> Co-authored-by: withyj-codeit <48437814+withyj-codeit@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
요구사항
기본
심화
주요 변경사항
스크린샷
멘토에게