Просьба критики верстки

[slipdem]

Добрый день. 

Просьба посмотреть на верстку и дать оценку/совет/критику по ней.

Подскажите, пожалуйста, что в моей верстке плохо, что хорошо а за что вообще по рукам давать надо.

Заранее спасибо! 

Всем хорошего дня

Макет тут

Верстка тут

[klierik]

имена классов через "--" я не совсем понял... и баги появляются при узкой ширине http://take.ms/CHh8G

#about div {
    width: 50%;
    height: 623px;
}

Лучше через классы всё... 

.price--plan--row {
    position: relative;
    margin-top: 77px;
}

Ну вот зачем такой класс?  и с одним "-" прочитать можно

font-size: 22.52px;

23px можно смело писать, ибо пол пикселя не бывает.

в целом слабенько, пока что

[SelenIT]

1 час назад, klierik сказал:

ибо пол пикселя не бывает.

Бывает, но не на всех платформах.

[slipdem]

Quote

имена классов через "--" я не совсем понял...

.price--plan--row {

Ну вот зачем такой класс?  и с одним "-" прочитать можно

Хорошо. Учту.

А в целом, что Вы скажете о названиях самих классов? Названия не очень ужасные?)

Quote

и баги появляются при узкой ширине http://take.ms/CHh8G

Мда, там таких багов - греблю гати. Буду разбираться. 

Лучше все через % и без фиксированной высоты?

Quote

23px можно смело писать, ибо пол пикселя не бывает.

Поставил пол пикселя, потому что так в шаблоне именно в этом месте было. Но спасибо - буду учитывать в будущем.

Quote

в целом слабенько, пока что

А в чем именно слабо? Что бы Вы посоветовали подтянуть и на что обратить внимание? 

Спасибо.

[klierik]

А почитай в подписи типичные ошибки...

[andrew312]

<a class="header--logo" href="index.html"><img src="images/logo.png" alt="Logo"></a>
.header--logo {
    float: left;
    height: 48px;
    width: 50px;
    padding-top: 22px;
    padding-bottom: 23px;
}

А как же тот факт, что свойства height, width применяются только к блочным элементам, элементам inline-block и рисункам? А у вас применяется к ссылке. Хотя почему-то работает

Логотип влияет на высоту header. Если поставить логотип чуть больше/меньше высота header изменится. Мне кажется удачней было бы сделать position:absolute для блока с логотипом.

Edited October 22, 2016 by andrew312

[andrew312]

<section id="bussines"> 
        </section>
#bussines {
min-width: 1170px;
}

Чтобы при уменьшении окна браузера фон и размер section не уменьшался, нужно добавить свойство.

[slipdem]

On 10/22/2016 at 8:27 PM, andrew312 said:

<a class="header--logo" href="index.html"><img src="images/logo.png" alt="Logo"></a>
.header--logo {
    float: left;
    height: 48px;
    width: 50px;
    padding-top: 22px;
    padding-bottom: 23px;
}

А как же тот факт, что свойства height, width применяются только к блочным элементам, элементам inline-block и рисункам? А у вас применяется к ссылке. Хотя почему-то работает

Логотип влияет на высоту header. Если поставить логотип чуть больше/меньше высота header изменится. Мне кажется удачней было бы сделать position:absolute для блока с логотипом.

1. Инлайн стает блочным если к нему применен флоат. Потому и работает.

2. Да, это надо будет исправить.

On 10/22/2016 at 9:04 PM, andrew312 said:

<section id="bussines"> 
        </section>
#bussines {
min-width: 1170px;
}

Чтобы при уменьшении окна браузера фон и размер section не уменьшался, нужно добавить свойство.

Спасибо