Skip to content
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

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

E-jiwoo
Copy link

@E-jiwoo E-jiwoo commented Jul 18, 2023

📄 Summary

마루 홈페이지의 메인 화면 디자인을 html과 css로 구현해보았다


🔨 Tasks

  • HTML
  • CSS

🙋🏻 More

image

E-jiwoo added 4 commits July 17, 2023 19:18
- 임시 사진으로 배경을 한 뒤 사진 위에 들어갈 요소들의 위치를 도트를 이용하여 잡아놓았다
- 배경에 사진을 추가하였고 글씨색을 바꾸어주었다
- 글씨, 버튼의 위치를 디자인과 동일하게 만들어주었다
- 사진위에 불투명 회색을 깔아줬고 모든 요소가 화면 중간에 배치되도록 하였다
@SEOKKAMONI SEOKKAMONI requested review from SEOKKAMONI and arkk200 July 18, 2023 06:08
@SEOKKAMONI SEOKKAMONI assigned SEOKKAMONI and E-jiwoo and unassigned SEOKKAMONI Jul 18, 2023
@arkk200 arkk200 assigned E-jiwoo and unassigned E-jiwoo Jul 18, 2023
Copy link
Member

@SEOKKAMONI SEOKKAMONI left a 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 Show resolved Hide resolved
styles/button.css Outdated Show resolved Hide resolved
styles/style.css Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
styles/button.css Outdated Show resolved Hide resolved
styles/style.css Outdated Show resolved Hide resolved
styles/style.css Outdated Show resolved Hide resolved
styles/style.css Outdated Show resolved Hide resolved
styles/style.css Outdated Show resolved Hide resolved
styles/text.css Outdated
Comment on lines 5 to 25
.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;
}
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

텍스트 별로 클래스를 나눈 점이 보기 좋네요👍

styles/style.css Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
index.html Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
E-jiwoo added 2 commits July 18, 2023 18:26
-코드 리뷰를 해주신 것을 토대로 코드를 수정함
-어둡게 되어 있는 이미지로 뒷배경 사진을 바꾸었다
@E-jiwoo
Copy link
Author

E-jiwoo commented Jul 18, 2023

수정하였습니다!

-justify-content에서 center로 지정해주었던 걸 space-between으로 바꾸어 주었다
@SEOKKAMONI SEOKKAMONI self-requested a review July 19, 2023 15:04
@hotsemiwest
Copy link
Member

아 개웃경 수고하셧어요

Copy link
Member

@SEOKKAMONI SEOKKAMONI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨어요 엄청 깔끔하네요 !!

styles/button.css Outdated Show resolved Hide resolved
styles/style.css Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
styles/header.css Outdated Show resolved Hide resolved
E-jiwoo added 2 commits July 20, 2023 01:52
-padding 값을 바꾸고 html 부분에 줬던 속성을 container에 주었으며 button 정렬에 position 대신 flex를 사용하였다
-text, container의 네이밍을 바꾸었다
@SEOKKAMONI SEOKKAMONI self-requested a review July 24, 2023 05:12
Copy link
Member

@SEOKKAMONI SEOKKAMONI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다~

Copy link

@arkk200 arkk200 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

과제한다고 수고 많으셧습니다

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants