Skip to content

파일추가#2

Open
Hyukjin-Ko wants to merge 2 commits intomainfrom
mini
Open

파일추가#2
Hyukjin-Ko wants to merge 2 commits intomainfrom
mini

Conversation

@Hyukjin-Ko
Copy link
Copy Markdown
Collaborator

No description provided.

Comment thread css/style.css
.main-box {
height: 430px;
width: 1425px;
background-image: url("../bg.jpeg");
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

현재 배경이 반복되어서 나오고 있는데 의도하신게 아니라면
background: url(../bg.jpeg) no-repeat center center; 이렇게 작성해보시는건 어떨까요?
no-repeat은 사진을 한번만 보이게 해줘요~

Comment thread index.html
<div class="main-box">
<div class="profile-box">
<div class="profile-picture">
<img src="programmer.jpeg" style="width:200px;border-radius:50%" >
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

이미지의 크기를 지정해줄 때 style.css파일 말고 html 태그의 style속성에 css를 작성해준 이유가 따로 있으실까요??

Comment thread css/style.css
float:left;
color:white;
font-size:25px;
margin:0 0 0 0;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

margin:0 0 0 0;은 margin:0과 같으므로 0 하나만 작성하시면 가독성이 더 좋을 거 같아요!

Comment thread css/style.css
}

nav a {
text-decoration: none;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

a태그의 text-decoration : none;를 99번째 라인에도 주셨는데 이렇게 반복적으로 작성된다면 body에 적용해주면 어떨까요?

Comment thread css/style.css

justify-content : center;
text-align: center;
font-family: "jack";
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

51, 61번째 라인에서도 중복되어 작성하셨는데 body로 옮겨 보시는 건 어떨까요?
body태그에서 한번만 작성하면 되니까 코드 유지 보수할 때 훨씬 수월할 거 같아요

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.

2 participants