-
Notifications
You must be signed in to change notification settings - Fork 1
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
[이지우] 과제 제출합니다 #1
base: main
Are you sure you want to change the base?
Conversation
- 임시 사진으로 배경을 한 뒤 사진 위에 들어갈 요소들의 위치를 도트를 이용하여 잡아놓았다
- 배경에 사진을 추가하였고 글씨색을 바꾸어주었다
- 글씨, 버튼의 위치를 디자인과 동일하게 만들어주었다
- 사진위에 불투명 회색을 깔아줬고 모든 요소가 화면 중간에 배치되도록 하였다
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.
과제하느라 수고하셨습니다 끝까지 포기하지않고 구현해온 부분이 너무 대견스럽네요
리뷰를 천천히 읽어보면서 코드를 수정하시면 될 것 같습니다 !
추가적으로 이제부터 개발하는 관점을 조금 바꾸시면 좋을것같아요
그냥 개발해서 구현한다 보다는 그 후 개발자가 좀 더 보기 좋게 아무나 와도 무슨 코드인지 읽기 쉽게 만든다를 생각하면서 코딩하다 보시면 더 좋은 개발자가 될 것 같아요 !
styles/text.css
Outdated
.text{ | ||
width: 457px; | ||
height: 188px; | ||
line-height: 93.6px; | ||
font-family: Pretendard,'Noto Sans KR', sans-serif; | ||
font-weight: 700; | ||
font-size: 72px; | ||
letter-spacing: -1.5px; | ||
color: #fff; | ||
} | ||
.text2{ | ||
height: 82px; | ||
padding-top: 24px; | ||
font-family: Pretendard,'Noto Sans KR', sans-serif; | ||
font-weight: 400; | ||
font-size: 18px; | ||
line-height: 28.8px; | ||
letter-spacing: -0.15px; | ||
vertical-align: bottom; | ||
color: #BBBCC2; | ||
} |
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.
텍스트 별로 클래스를 나눈 점이 보기 좋네요👍
-코드 리뷰를 해주신 것을 토대로 코드를 수정함
-어둡게 되어 있는 이미지로 뒷배경 사진을 바꾸었다
수정하였습니다! |
-justify-content에서 center로 지정해주었던 걸 space-between으로 바꾸어 주었다
아 개웃경 수고하셧어요 |
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.
수고하셨어요 엄청 깔끔하네요 !!
-padding 값을 바꾸고 html 부분에 줬던 속성을 container에 주었으며 button 정렬에 position 대신 flex를 사용하였다
-text, container의 네이밍을 바꾸었다
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.
과제한다고 수고 많으셧습니다
📄 Summary
🔨 Tasks
🙋🏻 More