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

Курсы §14. Мастерская: декоративные элементы
1 
<head>
    <meta charset="utf-8">
    <title>Испытание: карточка курса</title>
</head>
<body>
    <div class="main-block">
       <div class="text">
           <h2>Курс &laquo;Фоны&raquo;</h2>
            <p>Учимся использовать фоновые цвета и изображения, разбираемся с форматами, знакомимся с приёмами создания декоративных эффектов.</p>
            <div class="share">
                <a href="#facebook" class="social social-fb"></a>
                <a href="#twitter" class="social social-tw"></a>
                <a href="#google" class="social social-g"></a>
            </div>
       </div>
    </div>   
</body>

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;
}
.main-block {
    background: #1abc9c;
    width: 200px;  
    padding: 20px;
    padding-bottom: 20px;
    color: white;
}
.main-block h2 {
    font-weight: normal;
    margin-top: 0;
    margin-bottom: 20px;
    font-size: 20px
}
    
.share {
    width: 120px;
    margin-top: 25px;
}
.social {
    display: inline-block;
    width: 30px;
    height: 30px;
    margin-right: 5px;
    background-image: url("/assets/course14/social-challenge.png");
    background-repeat: no-repeat;
    background-color: #16a085;
    vertical-align: middle;
}

.social-fb {    
    background-position: 0 0;
}
.social-tw {
    background-position: -30px 0;
}
.social-g {
    background-position: -60px 0;
}
2

Всё в порядке. Только я бы не делал обёртку div.text (зачем она?) и в стилях .main-block непонятно для чего padding: 20px; (который задаёт паддинги со ВСЕХ сторон по 20) и сразу нижу - padding-bottom: 20px;

1 лайк
3

Лучше не ограничивать ширину .share.

1 лайк