[27/32] Испытание: строим сетку. Замечания по коду. Можно ли что-то упростить?


#1

Здравствуйте! Выполнил задание на 100%, но хотелось бы увидеть мнение старших товарищей. Все ли правильно написано и можно ли как-то упростить код? Заранее спасибо!!

HTML:

  <!DOCTYPE html>
    <html lang="ru">
        <head>
            <title>Испытание: строим сетку</title>
            <meta charset="utf-8">
        </head>
        <body>
            <div class="head">
                <div class="layout-positioner">
                    <div class="layout-column">Header</div>
                </div>
            </div>
            <div class="menu">
                <div class="layout-positioner">
                    <div class="layout-column">Menu</div>
                </div>
            </div>
            <div class="promo">
                <div class="layout-positioner">
                        <div class="layout-column">Promo 1</div>
                        <div class="layout-column">Promo 2</div>
                </div>
            </div>
            <div class="features">
                <div class="layout-positioner">
                    <div class="left layout-column">Left</div>
                    <div class="main layout-column">Main</div>
            <div class="right layout-column">Right</div>
        </div>
    </div>
    <div class="footer">
        <div class="layout-positioner">
            <div class="layout-column">Footer</div>
        </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;
}

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

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

.head,
.promo {
    background-color: #34495e;
    padding-top: 10px;
    padding-bottom: 10px;
}

.head .layout-column {
    background-color: #c0392b;
    min-height: 25px;
    padding: 5px;
}

.menu {
    background-color: #3498db;
}

.menu .layout-column,
.footer .layout-column {
    padding: 5px;
    min-height: 25px;
}

.promo .layout-column {
    float: left;
    margin-right: 10px;
    width: 165px;
    background-color: #c0392b;
    min-height: 55px;
    padding-top: 5px;
    padding-left: 5px;
}

.promo .layout-column:last-child {
    margin-right: 0px;
}

.features .layout-column {
    float: left;
    margin: 10px 10px 10px 0px;
    width: 65px;
    background-color: #3498db;
    min-height: 105px;
    padding-top: 5px;
    padding-left: 5px;
}

.features .layout-column:last-child {
    margin-right: 0px;
}

.features .main {
    width: 185px;
}

.footer {
    background-color: #34495e;
}

#2

Вопросов к коду не имею, всё грамотно.


#3

вроде 100% может будут какие нибудь замечания?

<!DOCTYPE html>
<html lang="ru">
    <head>
        <title>Испытание: строим сетку</title>
        <meta charset="utf-8">
    </head>
    <body>
        <div class="header clearfix">
            <div class="position">
            <div class="header-logo ">Header</div>
            </div>
        <div class="header-menu">Menu</div>
        <div class="position">
            <div class="promo1 ">Promo 1</div>
        <div class="promo2">Promo 2</div>
        </div>
        </div>
       <div class="position">
           <div class="left"> Left</div>
               <div class="main">Main</div>
                <div class="right">Right</div>
                </div>
                <div class="footer">
                  <div class="footer-1"> 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:#34495e
}
.position{
width: 350px;
margin: 0 auto;
padding: 10px 10px
}
.position::after {
content: “”;
display: table;
clear: both;
}
.header-logo {
float: left;
background-color:#c0392b;
height:25px;
padding: 5px;
width: 340px;
}
.header-menu {
background-color:#3498DB;
width:auto;
height:25px;
padding:5px 55px ;
}
.promo1 {
float:left;
background-color:#c0392b;
width:160px;
height:50px;
padding: 5px;
margin-right: 10px
}
.promo2{
float:left;
background-color:#c0392b;
width: 160px;
height: 50px;
padding: 5px;
}
.left {
float:left;
background-color:#3498DB;
width: 60px;
height: 100px;
padding: 5px;
margin:0;
margin-right:10px
}
.main {
float:left;
background-color:#3498DB;
width: 180px;
height: 100px;
padding:5px;
}
.right {
float: right;
background-color:#3498DB;
width: 60px;
height: 100px;
padding: 5px;
margin-left:10px
}
.footer {
background-color:#34495e
}
.footer-1 {
width:350px;
height:25px;
padding: 5px 55px

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


#4
<head>
    <title>Испытание: строим сетку</title>
    <meta charset="utf-8">
</head>
<body>
    <div class="header clearfix">
        <div class="position">
        <div class="header-logo ">Header</div>
        </div>
    <div class="header-menu">Menu</div>
    <div class="position">
        <div class="promo1 ">Promo 1</div>
    <div class="promo2">Promo 2</div>
    </div>
    </div>
   <div class="position">
       <div class="left"> Left</div>
           <div class="main">Main</div>
            <div class="right">Right</div>
            </div>
            <div class="footer">
              <div class="footer-1"> Footer</div>
    </div>
</body>

#5

Сравните свой код с кодом выше в этой же теме и задайте вопросы.


#6

Сравнил, отличия конечно есть , но итог же один и тот же)) или я чего то не понимаю, объясните если не трудно.


#7

Под словом “итог” вы понимаете визуальное соответствие.
Перечислите главные по вашему мнению отличия, и мы их обсудим.