Прошу оценить код, предложите ваши варианты улучшения.

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

Выполнил на 100%.



    
        Испытание: строим сетку
        
    
    
        

            

                

                    Header
                

        

                

                  

                    Menu
                  

                

            

                

                    

                        Promo 1
                    

                    

                        Promo 2
                    
 
                

            

        

            

                

                    Left
                

                

                    Main
                

                

                    Right
                

            

        

        

          

                Footer
          

        

        
    





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

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

.header {
    background: #34495e;
    height: 70px;
    padding-top: 10px;
}

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

.block, .column {
    padding: 5px;
}

.block-header {
    min-height: 25px;
    background: #c0392b;
    margin-bottom: 10px;
}

.block-menu {
    min-height: 25px;
    background: #3498DB;
    padding-left: 15px;
}

.block-promo-1 {
    min-height: 50px;
    background: #c0392b;
    float: left;
    width: 160px;
}

.block-promo-2 {
    min-height: 50px;
    background: #c0392b;
    float: right;
    width: 160px;
}

.content {
    background: none;
    height: 75px;
    margin-top: 10px;

}

.column-left {
    min-height: 100px;
    width: 60px;
    background: #3498DB;
    float:left;
}

.column-right {
    min-height: 100px;
    width: 60px;
    background: #3498DB;
    float:right;
}

.column-main {
    min-height: 100px;
    width: 180px;
    margin-right: 10px;
    margin-left: 10px;
    background: #3498DB;
    float:left;
    margin-bottom: 10px;
}



.footer {
    clear: both;
    min-height: 25px;
    background: #34495e;
    padding-left: 15px;
}

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

Html-код не показали. В css-коде clearfix не вижу, зато есть фиксированные высоты у блоков. Центровщик в меню и футере отсутствует/не работает.

3

Центровщик для меню и футера - исправил. Фиксированные высоты блоков это плохо?

4

Фиксированные высоты плохо, если текста станет больше - текст будет попросту “выпадать” из блока. поэтому, лучше использовать min-height - тогда при переполнении блок будет растягиваться