[27/32] Покритикуйте пожалуйста код

Курсы §15. Сетки
1

Здравствуйте, задание сделано на 100%, но складывается впечатление что сделано много лишних телодвижений, хотелось бы услышать критику и замечания к коду:

HTML:

<head>
    <title>Испытание: строим сетку</title>
    <meta charset="utf-8">
</head>
<body>
    <div class="fon clearfix">
        <div class="posit">
            <div class="hed">
                    Header
            </div>
        </div>
        <div class="men">
            <div class="posit">
                <div class="men-logo">
                    Menu
                </div>
            </div>
        </div>
        <div class="posit">
            <div class="prom1">
                Promo 1
            </div>
            <div class="prom2 ">
                Promo 2
            </div>
        </div>
    </div>
    <div class="posit">
        <div class="column column1">
            Left
        </div>
        <div class="column column2">
            Main
        </div>
        <div class="column column3">
            Right
        </div>
    </div>
    <div class="footer">
        <div class="posit">
            <div class="foot-logo">
                Footer
            </div>
        </div>
    </div>
</body>

CSS:

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

body {
width: 450px;
height: 335px;
font-family: “Arial”, sans-serif;
font-size: 10px;
color: white;
}

.posit{
width: 350px;
margin: auto;
}

.fon{
padding: 10px 0;
margin-bottom: 10px;
background: #34495e;
}

.hed{
width: 340px;
height: 25px;
padding: 5px;
background: #c0392b;
}

.men{
min-height: 35px;
margin: 10px 0;
background:#3498D8;
}

.men-logo{
padding: 5px;
}

.prom1,
.prom2{
width: 160px;
height: 50px;
padding: 5px;
float: left;
background: #c0392b;
}

.prom1{
margin-right: 10px;
}

.clearfix::after{
display: table;
content: “”;
clear: both;
}

.column{
min-height: 100px;
padding: 5px;
margin-right: 10px;
margin-bottom: 10px;
float: left;
background: #3498DB
}

.column:last-child{
margin-right:0;
}

.column1,
.column3{
width: 60px;
}

.column2{
width: 180px;
}

.footer{
min-height: 35px;
clear: both;
background: #34495e;
}

.foot-logo{
padding: 5px;
}

2

Не хватает две распорки, а там, где вы ее использовали, она не нужна.
Пишите min-height для блоков, сетка должна тянуться по вертикали, когда заполняется контентом.

3

Спасибо, по поводу min-height понял, по поводу распорок, я поставил распорку на “fon” для того чтобы из него не вылетели зафлоаченные prom1 и prom2 (далее не предполагалось чтобы fon продолжался, на фоне трех столбцов цвет от body, а footer имеет своё свойство цвета), соответственно куда две другие распорки ставить не совсем понимаю, поясните пожалуйста.

4

Распорки должны быть у .posit родителя двух колонок и трех колонок.