[27/32] Помогите улучшить код


#1

Подскажите, пожалуйста, есть ли какие-то недочеты в коде, как его улучшить и упростить? Заранее спасибо.

HTML
<!DOCTYPE html>
<html lang="ru">
    <head>
        <title>Испытание: строим сетку</title>
        <meta charset="utf-8">
    </head>
    <body>
       <div class="layout1">
            <div class="header layout-positioner"><p>Header</p>
            </div>
       </div>
        <div class="layout2"> 
            <div class="layout-positioner"> <p>Menu</p>
            </div>
        </div>
        <div class="layout3"> 
            <div class="layout-positioner">
               <div class="promo promo1"> <p>Promo 1</p>
               </div>
               <div class="promo promo2"> <p>Promo 2</p>
               </div>
            </div>
        </div>
        <div class="layout4 layout-positioner"> 
                <div class="columns left"> <p> Left </p>
                </div>
                <div class="columns main"> <p> Main</p>
                </div>
                <div class="columns right"> <p>Right</p>
                </div>
        </div>
        <div class="layout5"> 
            <div class="layout-positioner"> <p> Footer </p> 
            </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;
}

p {
    margin: 0;
    padding: 5px;
    
}
    
.layout-positioner {
    margin: auto;
    width: 350px
}
    
.layout1 {
    background-color: #34495e;
    min-height: 45px;
    padding-top: 10px;
}

.header {
    background-color: #c0392b;
    width: 350px;
    margin: 0 auto;
    min-height: 35px;
    
}

.layout2 {
    background-color: #3498DB;
    min-height: 35px;
}


 
.layout3 {
     background-color: #34495e;
     min-height: 70px;
     padding-top: 10px;
}
     
.promo {
    background-color: #c0392b;
    min-height: 60px;
    width: 170px;
    
}

.promo1 {
    float: left;
}

.promo2 {
    float: right;
}

.layout4 {
    min-height: 70px;
    padding-top: 10px;
}

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

.columns {
    background-color: #3498DB;
    min-height: 110px;
}

.left {
    float: left;
    margin-right: 10px;
    width: 70px;
}

.main {
    float: left;
    margin-right: 10px;
    width: 190px;
}

.right {
    float: right;
    margin-right: 0px;
    width: 70px;
}



.layout5 {
    background-color: #34495e;
    margin-top: 10px;
    min-height: 35px;
}

#2

Почему распорка только в блоке с тремя колонками?


#3

Все остальное отображалось корректно без распорки, поэтому решил не плодить сущности


#4

Распорка и была бы одна в css, у layout-positioner, но использовалась бы в каждом блоке, где есть этот класс.


#5

А зачем добавлять распорку туда, где она не нужна? В хэдере есть layout-positioner, но в распорке нет смысла


#6

аналогичная просьба - посмотреть, есть ли недочеты… (в конце html теги все закрыты - просто не дает их сюда корректно скопировать). Спасибо.

html
    <html lang="ru">
<head>
    <title>Испытание: строим сетку</title>
    <meta charset="utf-8">
</head>
<body>
<div class="header">  
    <div class="layout-positioner">
        <div class="top"> Header </div>
    </div>
        <div class="menu"> Menu </div>
    <div class="layout-positioner">
        <div class="Promo1">  Promo 1 </div>
        <div class="Promo2">  Promo 2 </div>
    </div>
</div>
    <div class="layout-positioner">
        <div class="left">  Left </div>
        <div class="main"> Main </div>
        <div class="right">  Right </div>
    </div>
    <div class="footer"> Footer </div>
CSS
html,

body {
margin: 0;
padding: 0;
}

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

.layout-positioner::after {
display: table;
content: “”;
clear: both;
}

.layout-positioner {

width: 350px;
margin: 0 50px;

/* border: 2px dashed #3498db; */
}

.header {
margin-bottom: 10px;
background: #34495e;
padding-bottom: 10px;
padding-top: 10px;

}

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

.menu {
margin-top: 10px;
min-height: 25px;
background: #3498DB;
padding: 5px 55px;
}
.Promo1 {
margin-top: 10px;
min-height: 50px;
background: #c0392b;
padding: 5px;
float: left;
width: 160px;
}
.Promo2 {
margin-top: 10px;
min-height: 50px;
background: #c0392b;
padding: 5px;
float: right;
width: 160px;
}

.left {

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

}

.main {

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

}
.right {

min-height: 100px;
background: #3498DB;
padding: 5px;
float: left;
width: 60px;

}
.footer {
margin-top: 10px;
min-height: 25px;
background: #34495e;
padding: 5px 55px;

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