[신혜윤] 풀스택-FE-mission2#7
Open
hyeyoonS wants to merge 27 commits intocodeit-bootcamp-nodejs:풀스택-FEfrom
Hidden character warning
The head ref may contain hidden characters: "\ud480\uc2a4\ud0dd-FE-mission2"
Open
Conversation
풀스택 fe-mission1
hyeyoonS
commented
May 28, 2024
Comment on lines
+167
to
+170
| <a | ||
| href="https://www.facebook.com/?locale=ko_KR" | ||
| target="_blank" | ||
| rel="noopener noreferrer" |
Collaborator
Author
There was a problem hiding this comment.
a태그에서 target_blank를 사용하면 보안에 취약할 수 있다고 해요!
참고 자료
그래서
<a
href="https://www.facebook.com/?locale=ko_KR"
target="_blank"
rel="noopener noreferrer"
...이렇게 써주었습니당
hyeyoonS
commented
May 28, 2024
| target="_blank" | ||
| rel="noopener noreferrer" | ||
| > | ||
| <img src="/src/assets/svg/facebook-icon.svg" alt="facebook_ogo" /> |
Collaborator
Author
There was a problem hiding this comment.
앗 셀프 코드리뷰 하다가 발견,,, 오타 + 다른 alt들과 규칙도 다르네요 ,,, 수정하겠습니다
hyeyoonS
commented
May 28, 2024
Comment on lines
+14
to
+18
| <link | ||
| rel="stylesheet" | ||
| type="text/css" | ||
| href="https://cdn.jsdelivr.net/gh/orioncactus/pretendard/dist/web/static/pretendard.css" | ||
| /> |
Collaborator
Author
There was a problem hiding this comment.
웹폰트를 불러오는 방법을 선택했는데 성능상 이 방식은 지양하는게 좋다고 하네요! 🥲
hyeyoonS
commented
May 28, 2024
Collaborator
Author
There was a problem hiding this comment.
signin과 signup의 CSS속성이 거의 같아서 하나의 파일로 관리하기로 했어요!
This file contains hidden or 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
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.
요구사항
기본 요구사항
공통
README.md파일을 작성합니다.스프린트 미션레포지토리를 fork 합니다.로그인 페이지, 회원가입 페이지 공통
로그인 페이지
회원가입 페이지
심화 요구사항
공통
로그인 페이지, 회원가입 페이지 공통
구현 스크린샷
QA종합 의견
난이도
분량
Mission2 구현 설명
의견
문의사항 (궁금궁금)
Q. Mission1 요구사항 ↔ Mission2 요구사항 불일치 문제
⇒ 경로가 통일되어야 할 것 같아요~!
Q. ReadMe에 어떤 걸 작성해야 하는지 구체적인 가이드가 있을까요? 제가 쓰고 싶은 내용을 쓰면 되나요….?
Q. 적절한 코멘트를 남겨달라는 요구사항이, 셀프 코드리뷰를 의미하는 것이 맞을까요....?