27на 100% ГО конструктивную КРИТИКУ кода

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

HTML код:

<!DOCTYPE html>
<html lang="ru">
    <head>
        <title>Испытание: строим сетку</title>
        <meta charset="utf-8">
    </head>
    <body>
        <div class="header clearfix">
            <div class="layout-positioner">
                <div class="header1">Header</div>
            </div>
            <div class="menu">
                <div class="layout-positioner">
                <span>Menu</span>
                </div>
            </div>
            <div class="layout-positioner">
                 <div class="promo">Promo 1</div>
                 <div class="promo">Promo 2</div>
            </div>
        </div>
        <div class="clearfix">
             <div class="layout-positioner">
                 <div class="layout-column">Left</div>
                 <div class="layout-column">Main</div>
                 <div class="layout-column">Right</div>
             </div>
        </div>
        <div class="footer">
            <div class="layout-positioner">
                 <span>Footer</span>
            </div>
        </div>
    </body>
</html>

CSS код:

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

body {
width: 450px;
height: 335px;
font-family: “Arial”, sans-serif;
font-size: 10px;
color: white;
}
.clearfix::after {
content: “”;
display: table;
clear: both;
}
.layout-positioner {
width: 350px;
margin: 0 auto;
}
.header {
background-color: #34495e;
padding-top: 10px;
}
.header1 {
background-color: #c0392b;
margin-bottom: 10px;
padding:5px;
height: 25px;
}
.menu {
background-color: #3498DB;
margin-bottom: 10px;
height: 25px;
padding:5px;
}
span {
padding: 5px;
}
.promo {
background-color: #c0392b;
margin-bottom: 10px;
float:left;
margin-right: 10px;
width: 160px;
height: 50px;
padding: 5px;
}
.promo:last-child {
margin-right: 0px;
}
.layout-column {
background-color: #3498DB;
float:left;
min-height: 100px;
width: 60px;
margin: 10px 0;
padding: 5px;
margin-right: 10px;
}
.layout-column:nth-child(2) {
width: 180px;
}
.layout-column:last-child {
margin-right: 0px;
}
.footer {
background-color: #34495e;
height: 25px;
padding: 5px;
}

2

html-код не вижу.

1 лайк
3

Чем можно улучшить код

<!DOCTYPE html>
<html lang="ru">
<head>
    <title>Испытание: строим сетку</title>
    <meta charset="utf-8">
</head>
<body>
    <div class="header">
        <div class="layout-positioner">
            <div class="layout-column-1 layout-column">Header</div>
            <div class="layout-column-2 layout-column">Menu</div>
            <div class="layout-column-3 layout-column">Promo 1</div>
            <div class="layout-column-4 layout-column">Promo 2</div>
        </div>
    </div>
    <div class="features">
        <div class="layout-positioner">
            <div class="layout-column-1">
                <div class="feature hidden">Left</div>
            </div>
            <div class="layout-column-2">
                <div class="feature hidden-1">Main</div>
            </div>
            <div class="layout-column-3">
                <div class="feature hidden">Right</div>
            </div>
        </div>
    </div>
<div class="footer">
    <div class="layout-positioner">
        <div class="layout-column">Footer</div>
    </div>
</div>
</body>
</html>

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

body {
width: 450px;
height: 335px;
font-family: “Arial”, sans-serif;
font-size: 10px;
color: white;
}
.header {
width: auto;
height:170px;
margin-bottom: 10px;
color: white;
background: #34495e;
}
.layout-positioner {
width: 430px;
margin: 0 auto;
}
.layout-positioner::after {
display: table;
content: “”;
clear: both;
}
.header .layout-column-1 {
float: left;
width: 340px;
height: 25px;
background: #c0392b;
margin: 10px 0px 10px 40px;
padding: 5px 5px 5px 5px;
}
.header .layout-column-2 {
float: left;
width: 395px;
height: 30px;
background: #3498DB;
margin-left: -10px;
padding: 5px 0px 0px 55px;
}
.header .layout-column-3 {
float: left;
clear: right;
width: 160px;
height: 50px;
background: #c0392b;
margin: 10px 5px 10px 40px;
padding: 5px 5px 5px 5px;
}
.header .layout-column-4 {
float: left;
width: 160px;
height: 50px;
background: #c0392b;
margin: 10px 0px 0px 5px;
padding: 5px 5px 5px 5px;
}
.features {
margin-bottom: 10px;
}

.features .layout-column-1 {
float: left;
margin: 0px 5px 10px 40px;
padding: 5px 5px 5px 5px;
width: 60px;
height: 100px;
background: #3498DB;
}
.features .layout-column-2 {
float: left;
margin: 0px 5px 10px 5px;
padding: 5px 5px 5px 5px;
width: 180px;
height: 100px;
background: #3498DB;
}
.features .layout-column-3 {
float: left;
margin: 0px 5px 10px 5px;
padding: 5px 5px 5px 5px;
width: 60px;
height: 100px;
background: #3498DB;
}

.footer .layout-column {
float: left;
width: 395px;
height: 30px;
background: #3498DB;
margin: -10px;
padding: 5px 0px 0px 55px;
color: white;
background: #34495e;
}
/*
Используемые цвета:
#34495e – мокрый асфальт
#c0392b – красный
#3498DB – синий
*/

4

У вас неправильно подобрана ширина центровщика, из-за чего появляется дополнительный отступ слева 40px, а также отрицательные отступы.
Не ограничивайте высоту блоков, иначе ваша сетка не будет тянуться по высоте при заполнении ее контентом.