Испытание: строим сетку - ревью кода

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

Всем привет. Прокомментируйте, пожалуйста. Возможно, есть замечания, где-то исправить или добавить код.

    <body>
        <div class="container">
            <div class="header">Header</div>
            <div class="menu">
                <div class="layout-positioner">
                    <div class="menu-box">Menu</div>
                </div>
            </div>
            <div class="layout-positioner">
                <div class="promo">Promo 1</div>
                <div class="promo">Promo 2</div>
            </div>
        </div>
        
        <div class="features">
            <div class="layout-positioner">
                <div class="layout-column">Left</div>
                <div class="layout-column">Main</div>
                <div class="layout-column">Right</div>
            </div>
        </div>
        
        <div class="footer">
            <div class="layout-positioner">
                <div class="footer-box">Footer</div>
            </div>
        </div>
    </body>

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

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

.container {
    width: 450px;
    height: 160px;
    background: #34495e;
    margin-bottom: 10px;
    padding-top: 10px;
}

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

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

.header {
    width: 340px;
    min-height: 25px;
    padding: 5px;
    background: #c0392b;
    margin: 0 auto 10px;
}

.menu {
    min-height: 35px;
    background: #3498DB;
    margin-bottom: 10px;
}

.menu-box {
    padding: 5px;
}

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

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

.features {
    margin-bottom: 10px;
}

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

.features .layout-column:nth-child(2) {
    width: 180px;
}

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

.footer {
    min-height: 35px;
    background: #34495e;
}

.footer-box {
    padding: 5px;
}
2

Высоту .container ограничили. Других замечаний нет.

3

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