27 задание, переделал, проверьте плз)


#1
<div class="header-wrapper">
  <div class="layout-positioner">
    <div class="header">
      Header
    </div>
  </div>
  <div class="menu-wrapper">
    <div class="layout-positioner">
      <div class="menu">Menu</div>
    </div>
  </div>
  <div class="layout-promo">
    <div class="layout-positioner">
      <div class="promo1 promo">
        Promo 1
      </div>
      <div class="promo2 promo">
        Promo 2
      </div>
    </div>
  </div>
</div>
<div class="content-wrapper">
  <div class="layout-positioner">
    <div class="column left">Left</div>
    <div class="column main">Main</div>
    <div class="column right">Right</div>
  </div>
</div>
<div class="footer-wrapper">
  <div class="layout-positioner">
    <div class="footer">Footer</div>
  </div>
</div>



html,

body {
margin: 0;
padding: 0;
}

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

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

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

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

.header {
background-color: #c0392b;
width: 350px;
height: 35px;
padding: 5px;
box-sizing: border-box;
}

.menu-wrapper {
background-color: #3498db;
height:35px;
margin-top: 10px;
}

.menu,
.promo,
.column,
.footer {
padding: 5px;
box-sizing: border-box;
}

.promo {
background-color: #c0392b;
width: 170px;
height: 60px;
margin-top: 10px;
margin-bottom: 10px;
}

.layout-promo::after {
content: “”;
display: block;
clear: both;
}

.promo1 {
float: left;
}

.promo2 {
float:left;
margin-left: 10px;
}

.column {
background-color: #3498db;
height: 110px;
margin-top: 10px;
margin-bottom: 10px;
margin-right: 10px;
}

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

.left {
float: left;
width: 70px;
}

.main {
float: left;
width: 190px;
}

.right {
float: left;
width: 70px;
}

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

.content-wrapper {
clear: both;

}

.footer-wrapper {
background-color: #34495e;
height: 40px;
}

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


#2

Почему у вас в коде два правила содержат clear: both?


#3

Не заметил, спасибо
А в остальном правильно?


#4

Вместо height лучше писать min-height. В остальном нормально.


#5

Дело в том, что я не знал, что такое min-height,
Хорошо, поищу, спасибо!!!