27/32 100% Оцените код пожалуйста. Нужно больше критики))


#1

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

body {
width: 450px;
height: 335px;

font-size: 10px;
font-family: “Arial”, sans-serif;
color: white;
}

/*
Используемые цвета:
#34495e – мокрый асфальт
#c0392b – красный
#3498db – синий
*/

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

.main-header {
box-sizing: border-box;
background-color: #c0392b;
width: 350px;
min-height: 35px;
margin: 0 auto;
margin-bottom: 10px;
padding-left: 5px;
padding-top: 5px;
}

.main-menu {
box-sizing: border-box;
background-color: #3498db;
height: 35px;
padding-left: 55px;
padding-top: 5px;
margin-bottom: 10px;
}

.promo1,
.promo2 {
box-sizing: border-box;
background-color: #c0392b;
padding-top: 5px;
width: 170px;
height: 60px;
}

.promo1 {
box-sizing: border-box;
float: left;
margin-left: 50px;
padding-left: 5px;

}

.promo2 {
box-sizing: border-box;
float: right;
margin-right: 50px;
padding-left: 5px;
margin-bottom: 10px;
}

.top-block {
margin-bottom: 10px;
}

.middle-block {
background-color: #ffffff;
padding-left: 50px;
margin-bottom: 10px;
}

.left-block,
.center-block,
.right-block {
box-sizing: border-box;
background-color: #3498db;
float: left;
margin-right: 10px;
height: 110px;
padding-left: 5px;
padding-top: 5px;
}

.left-block,
.right-block {
width: 70px;
}

.center-block {
width: 190px;
}

.main-footer {
box-sizing: border-box;
background-color: #34495e;
height: 40px;
padding-left: 55px;
padding-top: 5px;
}

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


#3

Курс, желательно, пройти внимательнее: “сетка” не центрируется, не тянется по вертикали.


#4

Скорее не критика а совет)
После изучения float сразу перейдите на flexbox. В начале будет сложно,но с flex в разы быстрее сверстайте сетку любой сложности+адаптив на раз два. Сейчас легких сеток почти нету…


#5

Спасибо за совет, сейчас как раз изучаю флексы. В теории вроде все просто, но на практике конечно приходится поломать голову)


#6

Благодарю за совет, и за критику.