[assignment#1] 코드리뷰

[seojisoosoo]

와우~! 세 문제 다 풀다니! 나는 앞으로 동길이를 '이멋쟁이'라고 부르기로 했어요!
Q2에서 이미지 hover했을 때 살짝 커지는 거 보고 감동... transform도 쓰다니.. 최고셔
전체적으로 넘넘 잘했다고 생각합니다! 스타일 줄 때, id만 사용을 하셨는데, 앞으로는 class도 예뻐해주세요ㅎㅎ

Q1

<div>멋쟁이 사자처럼 10기 백엔드 개발에 관심이 많습니다</div>
    <div>중앙대학교 산업보안학과 2학년으로, 최근에 복학했어요!</div>
    <div>00년생, 23살이에요 앞으로 잘부탁드려요!</div>

• html에서 줄을 바꾸고 싶을 때, 각 줄마다 div를 사용할 수도 있지만, 보통 p태그나 br태그를 이용합니다! 지금 div쓴 거 대신에 p태그를 사용해도 좋고, br태그 한 번을 써서 강제로 줄바꿈을 시킬 수도 있어요!

Q3

#model_s, #model_x{
    font-size: 30px;
    transition : transform 1s;
    margin-left: 30px;
    text-align: center;
}
#model_s:hover, #model_x:hover{
    transform:scale(1.2);
}

• 두 개의 id가 동일한 스타일을 주는 것이라면, id보다 class로 선언하는 것이 좋을 것 같아요! 보통 id는 한 번 불러올 스타일, class는 여러번 불러올 스타일에 적용을 하는데, 이 경우에는 스타일이 두 번 적용되는 것이기 때문에 class로 주는 것을 추천합니다! class로 선언하면, css에서는 #이 아니라 .을 이용합니다!

#white{
    margin: 0 2px 0 2px;
    width :15px;
    height: 15px;
    background-color: white;
    border-style:solid;
    border-width: 2px;
}
#red{
    margin: 0 2px 0 2px;
    width :15px;
    height: 15px;
    background-color: red;
    border-style:solid;
    border-width: 2px;
}

• 이 경우도, background-color를 제외한 스타일은 모두 동일하기 때문에, 여러번 불러오는 것보다, 이 스타일을 하나의 class로 주고, background-color만 id로 주면 좀 더 효율적인 코드가 될 것 같습니다!!

Q4

img{
    margin-top: 10px;
    margin-left: 15px;
}

• 넘 잘해서 리뷰할 게 없긴 하지만! 이 문제의 경우, 이미지를 하나밖에 사용하지 않았지만, 실제로 웹페이지를 만들 때는 여러 이미지를 많이 사용하게 될 테니, 그 때는 이미지에도 id나 class로 스타일 적용하면 좋을 것 같습니다!