-
Notifications
You must be signed in to change notification settings - Fork 21
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
[정성현] sprint3 #56
The head ref may contain hidden characters: "Basic-\uC815\uC131\uD604-sprint3"
[정성현] sprint3 #56
Conversation
- 'row-reverse'를 통한 요소 순서 결정 - 새 창에서의 기존 창 접근 방지를 위해 <a> 요소에 'rel="noopener noreferrer" 속성 추가 - <form> 요소에 적절한 method, type, name 속성 할당 - 초기화/전역CSS/CSS변수를 구분하도록 CSS 구조 변경 및 관련 파일 경로 변경
- 클래스 네이밍 규칙에 BEM(with camel) 방식을 적용함 - 변경된 클래스 네이밍 규칙에 맞춰 HTML/CSS 수정 - 추가적으로, form 내 field 구성 방식 통일
- 배포 경로를 변경함
- 1199px 이하(태블릿), 767px 이하(모바일)에 맞춰 구조, 여백, 글씨 크기가 달라지도록 CSS 작성
- 메인 페이지에 대한 오픈그래프를 작성함
- 767px 이하(모바일)에 맞춰 여백, 글씨 크기가 달라지도록 CSS 작성 - 구조를 위해 main 안에 main__contentWrapper 추가
<a class="head-login" href="/login.html">로그인</a> | ||
/> | ||
</a> | ||
<a class="header__loginButton" href="/login.html">로그인</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.
클래스 명 내에서 Camel Case + __
를 사용하셔서 더 명확하게 읽히네요. 너무 좋습니다! 👍
<a href="/" | ||
><img | ||
class="head-logo" | ||
<body class="body"> |
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.
HTML 문서 내에서 Body Element는 하나만 존재할 수 있어서 클래스로 구분하지 않으셔도 괜찮을 것 같습니다!
|
||
input { | ||
.formField__input { |
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.
TextField, Button 등의 순수한 공통 UI 요소는 별도의 디자인 시스템 CSS 파일로 분리하여 사용하셔도 좋을 것 같아요~!
물론 이제 React에서 컴포넌트 기반으로 개발하시기 때문에 하지 않으셔도 크게 지장은 없지만,
예전에 사용되던 Bootstrap의 방식과 유사하셔서 공유드립니다. :)
<li><a href="/privacy.html">Privacy Policy</a></li> | ||
<li><a href="/faq.html">FAQ</a></li> | ||
<footer class="footer"> | ||
<address class="footer__address">©codeit - 2024</address> |
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.
시멘틱 태그를 잘 활용하시네요. 좋습니다. 👍
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.
지금은 줄 바꿈 문자를 삽입하기 어려우셔서 줄바꿈 처리가 어려우실 수 있는데요.
추후에는 줄바꿈이 필요한지 여부에 따라 공백 문자를 white-space
를 normal 또는 pre-wrap로 나누어 처리하시면, 반응형을 더 쉽게 처리하실 수 있습니다. :)
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.
관련해서 토스 랜딩 페이지가 어떻게 하고 있는지 참고하시면 좋을 것 같아 공유드립니다~~!
(중간에 헤드라인 내에 줄바꿈 있는 요소를 개발자 도구로 보시면 됩니다.)
type="password" | ||
placeholder="비밀번호를 다시 한 번 입력해 주세요" | ||
/> | ||
<button class="formField__toggleButton" type="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.
제출 버튼이 아닌 경우 type으로 구분 잘 하셨습니다. 👍
웹 접근성을 고려한 요소 은폐 | ||
*/ | ||
|
||
.blind { |
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.
웹 접근성까지 고려하신 것 너무 좋습니다! 👍
다만 스타일 초기화보다는 유틸리티 클래스에 가까운 것 같아, 분리하여 관리해주시면 더욱 좋을 것 같습니다!
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.
아 초기화 파일이 아니라 전역 스타일 파일이었군요. 제가 잘못 봤습니다.
유틸리티 클래스가 많은게 아니면 아직은 그대로 쓰셔도 문제는 없을 것 같습니다. :)
@import url("/styles/common/variable.css"); | ||
@import url("/styles/common/global.css"); |
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.
순수한 CSS 파일 내에서 @import
구문으로 다른 CSS를 불러올 때는 병렬이 아니라 직렬로 불러오기 때문에 최적화 이슈가 있는데요.
다만, 실제로 웹 서비스를 만들 때 함께 사용하는 Webpack, Rollup 등의 번들러에서 이러한 구문을 <head/>
태그 내 <link/>
또는 <style/>
등으로 변환하거나 합쳐서 삽입해주기 때문에 문제는 없습니다.
번들러가 처리해주지 않는 경우 문제가 될 수 있다는 것만 아시면 문제 없을 것 같습니다!
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.
다음은 번들러 Rollup 내의 옵션과 이를 위해 사용되는 PostCSS의 플러그인입니다.
지금은 참고로만 알아두세요~! :)
margin-left: 180px; | ||
text-align: right; | ||
} | ||
@media (max-width: 1199px) { | ||
.card:nth-child(2n) { |
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.
선택자 활용하시는 것 너무 좋네요! 👍
모바일 디바이스에서도 정상 노출되도록 반응형 작업을 너무 잘 해주셨습니다. 이미 관련된 메타 태그까지 처리해주셨네요. 👍 성현님도 한 달 동안 너무 잘 해주셨습니다. 남은 파트도 잘 마무리하셔서 원하는 결과 얻어 가셨으면 좋겠습니다. 화이팅입니다! 👍 |
요구사항
기본
공통
랜딩 페이지
로그인, 회원가입 페이지 공통
심화
주요 변경사항
스크린샷
메인 페이지 모바일 뷰
로그인 페이지 PC 뷰
로그인 페이지 모바일 뷰(750px)
로그인 페이지 모바일 뷰(350px)
오픈그래프
멘토에게
(현재 text-wrap: balance; word-break: keep-all; overflow-wrap: break-word; 사용중입니다)