-
Notifications
You must be signed in to change notification settings - Fork 26
[안연정] sprint2 #78
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
[안연정] sprint2 #78
The head ref may contain hidden characters: "Basic-\uC548\uC5F0\uC815-sprint2"
Conversation
|
추가로, 추후 react와 같은 프레임워크를 도입하면 HTML, CSS, JS가 하나의 컴포넌트 안에 묶이는 구조로 바뀝니다. 스타일도 모듈화되거나 styled-components처럼 JS 안에서 함께 관리할 수 있어요. 또한 모든 페이지가 하나의 앱 내부에서 이동되기 때문에 index.html은 보통 하나만 존재합니다. 지금은 위 폴더구조를 참고해서 전체적인 구조를 이해해주시면 충분할 것 같습니다. 추후 프레임워크를 배우시면 자연스럽게 새로운 구조에 익숙해지실거에요. 화이팅입니다! 💪 |
| <div class="login-signup-main-logo"> | ||
| <img src="./img/login_page_logo.png" alt="판다마켓 로고"> | ||
| </div> | ||
| <div class="login-content"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
form 요소 구현 시에는 <form> 태그를 활용해주시면 좋습니다.
| <div class="login-content"> | ||
| <div class="member-info"> | ||
| <div class="login-email"> | ||
| <p>이메일</p> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
라벨 요소는 <label> 태그를 활용해주세요!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
또한 label 태그의 for 속성을 이용하면 input과 연동이되어 접근성이 좋아집니다.
<label for='email'>
| </div> | ||
| </div> | ||
| <div class="go-login"> | ||
| <button> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
form 태그 내부의 제출 버튼에 <button type='submit'> 을 추가해주면 input들과 연동이 되어 접근성을 개선할 수 있습니다.
| referrerpolicy="no-referrer"/> | ||
| </head> | ||
| <body> | ||
| <main class="signup-page"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
signup-page > signup-container > signup-content처럼 레이아웃을 구조적으로 잘 나누어주셨습니다👍
| <div class="signup-member-info"> | ||
| <div class="login-email"> | ||
| <p>이메일</p> | ||
| <input id="email" type="text" autoComplete="off"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
autocomplete 속성도 활용해보셨군요! 👍HTML 속성은 대소문자를 구분하지 않긴하지만, HTML 표준 or 린터에서 경고가 날 수 있어 일관되게 소문자(autocomplete)로 작성하는 게 좋습니다.
| 간편 로그인하기 | ||
| </div> | ||
| <div class="social-login-img"> | ||
| <a href="https://www.google.com/"><img src="./img/social-login-google.png"></a> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
alt 속성이 누락되었네요!
| <meta charset="utf-8"> | ||
| <title>판다마켓</title> | ||
| <link rel="stylesheet" href="style.css"> | ||
| <link |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
font awsome을 활용해주셔도 상관없긴하지만, 그러면 image asset 으로 등록한 눈 아이콘은 불필요해질 것 같아요. 불필요한 코드 or 이미지는 제거해주시면 좋겠습니다.
| position: fixed; /* 상단에 붙이기 */ | ||
| top: 0px; | ||
| left: 0px; | ||
| z-index: 1; /* 앞으로 빼기, 이것은 숫자 무엇으로 설정해야 좋을까? */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
실무에서는 영역별로 의미 있는 숫자 범위를 나눠서 유지보수 하기 쉽게 관리하는 편입니다.
예를 들어 기본 콘텐츠는 0-10, 헤더나 푸터는 100, 드롭다운이나 모달은 500, 알림창 로딩창은 999,1000 이상으로요!
요구사항
기본
공통
로그인 페이지
회원가입 페이지
심화
주요 변경사항
스크린샷
netlify 배포 주소


멘토에게
텍스트 중앙 정렬
텍스트 블록은 정중앙인데, 텍스트는 블록 내 중앙이 아닌 것 같은데 조정할 수 있는 방법이 궁금합니다.
프로젝트 구조
현재
html파일은 여러 개,css파일은 하나인데, 이 구조가 맞는지추후 프레임워크가 도입되면 프로젝트 구조는 보통 어떻게 만들어지는지 궁금합니다.