[한고은] sprint2#29
Hidden character warning
Conversation
greatelv
left a comment
There was a problem hiding this comment.
전반적인 코드 구조 개선 코멘트
UI 구현 및 컴포넌트 모듈화 등 작업하시느라 고생 많으셨습니다.
앞으로의 코드 품질을 높이고 더 나은 구조를 잡기 위해 세부 파일별로 알아두시면 좋은 핵심적인 팁들을 라인 코멘트로 남겼습니다. 시간 나실 때 한 번 확인해 주시기 바랍니다.
| flex-shrink: 0; | ||
| } | ||
|
|
||
| #logo { |
There was a problem hiding this comment.
스타일링 시 ID 대신 Class 선택자를 사용하는 것을 권장합니다.
#logo, #login 등 요소에 ID 선택자를 다수 사용하셨습니다. ID 선택자는 CSS 명시도(우선순위)가 매우 높기 때문에 추후 덮어쓰거나 다른 화면에서 동일한 스타일을 재사용하기 상당히 까다롭습니다. 가급적 .logo, .btn-large 와 같이 Class 기반으로 묶어 스타일링하는 구조를 지향해 주시면 추후 컴포넌트 관리가 훨씬 수월해집니다.
| 비밀번호</label> | ||
| <input class="form_input" type="password" name="password" placeholder="비밀번호를 입력해주세요"> | ||
| </div> | ||
| <a id="btn_large" href="">로그인</a> |
There was a problem hiding this comment.
폼(Form)을 전송하는 제출 버튼은 앵커(<a>) 태그보다는 <button type="submit">을 사용하는 것을 권장합니다.
유저가 정보를 입력한 뒤 엔터(Enter) 키를 쳤을 때 자연스레 폼이 제출되게 하기 위해서는 <button type="submit">을 사용하는 것이 시맨틱(Semantic) 표준에 알맞고 사용자 편의성 측면에서도 낫습니다.
|
|
||
| .section { | ||
| display: flex; | ||
| padding: 138px 344px; |
There was a problem hiding this comment.
화면 해상도를 고려하여 반응형 레이아웃 정렬을 사용해 보시는 걸 권장합니다.
현재 .section 영역 좌우에 padding: 138px 344px; 과 같이 큰 폭의 고정 픽셀을 주셨습니다. 이렇게 되면 좁은 화면의 노트북 화면 등에서는 남은 너비가 부족해져 콘텐츠 영역이 어색하게 줄어들 수 있습니다. 고정폭 여백 대신 요소에 max-width로 최대 폭을 제한하고, margin: 0 auto;를 부여해 상황에 맞춰 화면 가운데로 배치되는 방식(반응형 구조)을 고려해 보시기 바랍니다.
| </nav> | ||
| </header> | ||
| <!--HERO--> | ||
| <hero class="hero"> |
There was a problem hiding this comment.
커스텀 태그 지양 및 표준 시맨틱(Semantic) 마크업 활용을 권장합니다.
문서 내에 <hero>, <bottom>처럼 임의로 명명한 태그명들이 확인됩니다. 브라우저가 화면을 그리는 데는 문제가 없으나 웹 표준에 맞지 않아 SEO(검색엔진 최적화) 관점에서 인식되지 않습니다. 블록 단위를 구분하고 싶으시다면 <section class="hero"> 또는 <div> 구역 태그로 감싸주시는 것이 더 안전하고 좋은 습관입니다.
판다마켓 링크 : tubular-jalebi-bd5f2a.netlify.app
요구사항
스프린트 미션 1
기본 요구사항
랜딩 페이지
심화 요구사항
공통
reset.css를 설정해 주세요.
사용자의 브라우저 설정에 따라 기본 폰트 크기 설정이 변화함에 따라서 페이지의 요소 간 간격, 요소의 크기, font-size 등 모든 크기와 관련된 값이 크고 작아지도록 설정해 주세요.
스프린트 미션 2
기본 요구사항
공통
README.md 파일을 작성해 주세요.
마크다운 언어를 숙지하여 작성해 주세요.
내용은 자유롭게 작성해 주세요.
본인 브랜치(ex)part1-홍길동)에 스프린트 미션을 업로드해 주세요.
적절한 커밋 메시지를 남겨 주세요.
1-Sprint-Mission 레포지토리를 fork 합니다.
GitHub에 PR(Pull Request)을 생성해 upstream의 본인 브랜치(ex)part1-홍길동)에 미션을 제출합니다.
Git 활용 과정에서 유닉스 커맨드를 활용해 주세요.
HTML, CSS 파일을 Netlify로 배포합니다. (참고: https://www.codeit.kr/learn/5309)
로그인 페이지, 회원가입 페이지 공통
"판다마켓" 로고 클릭 시 루트 페이지("/")로 이동합니다.
SNS 아이콘들은 클릭 시 각각 "https://www.google.com/", "https://www.kakaocorp.com/page/" 으로 이동합니다.
input 요소에 focus out 일 때, 테두리는 없습니다.
input 요소에 focus in 일 때, 테두리 색상은 ##3692FF입니다.
로그인 페이지
"회원가입"버튼 클릭 시 "/signup" 페이지로 이동합니다.
회원가입 페이지
"로그인"버튼 클릭 시 "/login" 페이지로 이동합니다.
심화 요구사항
공통
로그인 페이지, 회원가입 페이지 공통
멘토에게