Оцените пожалуйста код Испытание 9/18


#1
<body>
    <div class="course-fones">
    <p class="header">Курс &laquo;Фоны&raquo;</p>
      <p>Учимся использовать<br> фоновые цвета и<br> изображения, разбираемся с форматами, знакомимся с приёмами создания декоративных эффектов.</p>
    <div class="share">
        <a class="social fb" href="#"></a>
        <a class="social tw" href="#"></a>
        <a class="social gl" href="#"></a>
    </div>
    </div>
</body>

CSS
html,
body {
margin: 0;
padding: 0;
}

body {
width: 280px;
min-height: 280px;
padding-top: 20px;
padding-left: 20px;
font-size: 14px;
font-family: “Arial”, sans-serif;
line-height: 1.2;
background: #ecf0f1;
}
.course-fones {
display: inline-block;
width: 220px;
height:235px;
margin: 0px;
padding-left: 20px;
text-align: middle;
color: white;
background-color: #1abc9c;
}
.share {
margin-top: 25px;
}
.social {
display: inline-block;
vertical-align: middle;
width: 30px;
height: 30px;
margin-right: 5px;
margin-bottom: 5px;
background-color: #16a085;
background-image: url("/assets/course14/social-challenge.png");
background-repeat: no-repeat;
}
.course-fones .header {
font-size: 20px;
}

.fb {
background-color: #16a085;
background-position: 0 0;
}

.tw {
background-color: #16a085;
background-position: -30px 0;
}

.gl {
background-color: #16a085;
background-position: -60px 0;
}


#2

Семантически лучше тег для заголовка h1 подойдет
и br попробуйте убрать и сделать переносы за счет отступов

это можно вынести в общий класс


#3
.course-fones {
    display: inline-block;    /* Зачем? */

#4

еще тут


#5

Лишнее. Но, не лишнее если иконки будут разные по высоте.


#6

Спасибо!