[27/32] Прошу указать на недостатки верстки

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

 <!DOCTYPE html>
<html lang="ru">
<head>
    <title>Испытание: строим сетку</title>
    <meta charset="utf-8">
</head>
<body>
    <div class="block1">
        <div class="center-block"><div class="header">Header</div></div>
        <div class="menu"><div class="center-block">Menu</div></div>
        <div class="center-block clearfix">
            <div class="promo1">Promo 1</div>
            <div class="promo2">Promo 2</div>
        </div>
    </div>
    
    <div class="block2">
        <div class="center-block clearfix">
            <div class="left">Left</div>
            <div class="main">Main</div>
            <div class="right">Right</div>
        </div>
    </div>
    <div class="block3">
        <div class="center-block">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;
}

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

.menu {
background: #3498DB;
padding: 5px;
padding-left: 15px;
min-height: 25px;
margin-bottom: 10px;
}

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

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

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

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

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

.block1 {
background: #34495e;
padding-top: 10px;
}

.block2 {
background: #fff;
padding-top: 10px;
}

.block3 {
background: #34495e;
padding: 5px;
padding-left: 15px;
height: 25px;
}

.center-block {
margin-left: auto;
margin-right: auto;
width: 350px;
}

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

2

Хороший код. Всё достаточно грамотно. Можете дополнительно поработать над объединением правил и свойств в css. А в целом замечаний нет.

1 лайк