Испытание: карточка курса [9/18]. 100%. Прокомментируйте, пожалуйста.


#1

HTML:

<!DOCTYPE html>
<html lang="ru">
    <head>
        <meta charset="utf-8">
        <title>Испытание: карточка курса</title>
    </head>
    <body>
        <div class="container">
            <h1>Курс &laquo;Фоны&raquo;</h1>
            <p>Учимся использовать фоновые цвета и изображения, разбираемся с форматами, знакомимся с приёмами создания декоративных эффектов.</p>
            <div class="social-container">
                <a class="social social-fb" href="#"></a>
                <a class="social social-tw" href="#"></a>
                <a class="social social-gp" href="#"></a>
            <div>    
        </div>
    </body>
</html>

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;
}

.container {
    box-sizing: border-box;
    width: 240px;
    height: 235px;
    background-color: #1abc9c;
    color: #ffffff;
    padding: 5px 20px;
}

.container h1 {
    padding: 0;
    margin-top: 15px;
    margin-bottom: 20px;
    font-weight: normal;
    font-size: 20px;
}

.container p {
    margin-bottom: 25px;
}


.social {
    display: inline-block;
    width: 30px;
    height: 30px;
    background-color: #16a085;
    background-image: url(/assets/course14/social-challenge.png);
    background-repeat: no-repeat;
    margin-right: 5px;
}

.social-fb {
    background-position: 0 0;
}

.social-tw {
    background-position: -30px 0;
}

.social-gp {
    background-position: -60px 0;
}

#2

Лучше для .container задать min-height, чтобы элемент растягивался под более длинный текст, но не схлопнулся, если текст окажется слишком коротким. А для последнего элемента в .social обнулите правый отступ. Если их будет не три, а пять, справа останется лишняя пустота и все может поехать