Оцените код пожалуйста)))

Испытание: строим сетку
    <div class="header">
        <div class="header-center">
            Header
        </div>
    </div>
    <div class="content">
        <div class="menu">
            Menu
        </div>
        <div class="doublepromo">
            <div class="promo1">
                Promo 1
            </div>
            <div class="promo2">
                Promo 2
            </div>
        </div> 
        <div class="triple">
            <div class="left">
                Left
            </div>    
            <div class="main">        
                Main
            </div>
            <div class="right">
                Right
            </div>
        </div>    
    <div class="footer">
        Footer
    </div> 
    </div>
</body>

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

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

.header{
background: #34495e;
height: 35px;
padding: 10px;
}

.header-center{
background: #c0392b;
height: 25px;
padding: 5px;
margin: 0 40px;
}

.menu{
background: #3498DB;
height: 25px;
padding: 5px 55px;
}

.doublepromo{
background: #34495e;
height: 80px;
}

.promo1{
float: left;
background: #c0392b;
width: 160px;
height: 50px;
padding: 5px;
margin-top: 10px;
margin-left: 50px;
}

.promo2{
float: right;
background: #c0392b;
width: 160px;
height: 50px;
padding: 5px;
margin-top: 10px;
margin-right: 50px;
}

.triple{
background: white;
height: 110px;
padding: 10px 50px;
}

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

.main{
float: left;
background: #3498DB;
width: 180px;
height: 100px;
padding: 5px;
}

.right{
float: right;
background: #3498DB;
width: 60px;
height: 100px;
padding: 5px;
}

.footer{
background: #34495e;
height: 25px;
padding: 5px 55px;
}

там 100% совпадение, но мне про семантику, качество, где там муть какая у меня

А что тут может быть не то? Всего то 5 строчек кода. Единственно, если я бы писал короткое margin: 50 px 70 px; то писал бы везде в документе.А не где то margin-left, а где то короткое margin. Хотя не критично, вообще

из недочетов: отсутствие в коде центровщика, псевдораспорок и использование фиксированной высоты - посмотрите другие темы на форуме по этому задание, обсуждается регулярно

2 лайка

Точно

<!DOCTYPE html>
<html lang="ru">
<head>
    <title>Испытание: строим сетку</title>
    <meta charset="utf-8">
</head>
<body>
    <div class="header">
        <div class="layout-positioner">
            <div class="header-center">
                Header
            </div>
        </div>
    </div>
    <div class="menu">
        <div class="layout-positioner">
            <div class="menu1">
                Menu
            </div>
        </div>
    </div>
    <div class="doublepromo">
        <div class="layout-positioner">
            <div class="promo1">
                Promo 1
            </div>
            <div class="promo2">
                Promo 2
            </div>
            <div class=clearfix>
            </div>
        </div>
    </div>
    <div class="triple">
        <div class="layout-positioner">
            <div class="left">
                Left
            </div>
            <div class="main">
                Main
            </div>
            <div class="right">
                Right
            </div>
            <div class=clearfix>
            </div>
        </div>
    </div>
    <div class="footer">
        <div class="layout-positioner">
            <div class="footer1">
                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 {
background: #34495e;
padding: 10px 0;
}

.header-center {
background: #c0392b;
min-height: 25px;
padding: 5px;
}

.menu {
background: #3498DB;
}

.menu1 {
padding: 5px;
min-height: 25px;
}

.doublepromo {
background: #34495e;
padding: 10px 0;
}

.promo1 {
float: left;
background: #c0392b;
width: 160px;
min-height: 50px;
padding: 5px;
}

.promo2 {
float: right;
background: #c0392b;
min-height: 50px;
width: 160px;
padding: 5px;
}

.triple {
background: white;
padding: 10px 0;
}

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

.main {
float: left;
background: #3498DB;
width: 180px;
min-height: 100px;
padding: 5px
}

.right {
float: right;
background: #3498DB;
width: 60px;
min-height: 100px;
margin-left: 0px;
padding: 5px;
}

.footer {
background: #34495e;
}

.footer1 {
min-height: 25px;
padding: 5px;
}

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

.layout-positioner {
width: 350px;
margin-left: auto;
margin-right: auto;

}

Вот так исправил, гляньте ещё плиз:slight_smile:

Кнопка для корректного отображения кода находится в редакторе сообщений и выглядит как </>.
Центровщик должен быть универсальным, то есть с автоматическими отступами справа и слева.

1 лайк

Если вы используете псевдораспорку, то класс clearfix добавляется родителю элементов со свойством float, у вас по коду это .layout-positioner.
Пустые блоки в разметке - это нехорошо.
В остальном код хороший.

<!DOCTYPE html>
<html lang="ru">
    <head>
        <title>Испытание: строим сетку</title>
        <meta charset="utf-8">
    </head>
    <body>
        <div class="container clearfix">
            <div class="layout-positioner">
                <div class="header"> Header </div>
            </div>
        </div>
            <div class="menu">Menu</div>
        <div class="container clearfix">   
            <div class="layout-positioner">
                <div class="column column-1"> Promo 1 </div>
                <div class="column column-2">Promo 2</div>
            </div>
        </div>  
        <div class="features clearfix">
            <div class="layout-positioner">
            <div class="features left"> Left </div>
            <div class="features main"> Main </div>
            <div class="features right"> Right </div>
            </div>
        </div>
        <div class="footer clearfix">Footer</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;
}
.container {
background-color: #34495e;
}
.header {
width: 345px;
height:30px;
background-color: #c0392b;
border: 10px solid #34495e;
margin-left: 40px;
padding-top: 5px;
padding-left: 5px;
}
.menu {
width: 395px;
height:30px;
background-color: #3498DB;
padding-top: 5px;
padding-left: 55px;
}

.column-1 {
float:left;
width: 160px;
height: 50px;
background-color: #c0392b;
margin-left: 50px;
margin-right:10px;
padding: 5px;
margin-bottom: 10px;
margin-top: 10px;
}
.column-2 {
float:left;
width: 160px;
height: 50px;
background-color: #c0392b;
padding: 5px;
margin-bottom: 10px;
margin-top: 10px;
}
.clearfix::after {
content: “”;
display: table;
clear: both;
}
.layout-positioner::after {
content:"";
display:table;
clear:both;
}

/*
.wrapper:first-child {
padding-top:5px;
}*/
p {
padding:5px;
}
.menu p {
padding-left:55px;
}
.features {
background-color:white;

}
.features .left {
float:left;
width:60px;
background-color:#3498DB;
margin-right:10px;
margin-top:10px;
height: 100px;
margin-left:50px;
padding:5px;
margin-bottom: 10px;
}
.features .right {
float:left;
width:60px;
background-color:#3498DB;
margin-right:0;
margin-top:10px;
height: 100px;
padding:5px;
margin-bottom: 10px;
}
.features .main {
width:180px;
background-color:#3498DB;
float:left;
margin-right:10px;
margin-top:10px;
height: 100px;
padding:5px;
margin-bottom: 10px;
}

.footer {
width: 395px;
height: 30px;
background-color:#34495e;
padding-left:55px;
padding-top:5px;
}

`Впишите сюда текст; также, отключить форматирование текста можно, начав строку с 4х пробелов`

Посмотрите код выше в этой теме. Сравните со своим, задайте конкретные вопросы по выявленным различиям.