Jump to content

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


slipdem
 Share

Recommended Posts

Добрый день. 

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

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

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

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

Макет тут

Верстка тут

Link to comment
Share on other sites

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

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

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

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

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

font-size: 22.52px;

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

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

Link to comment
Share on other sites

Quote

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

.price--plan--row {

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

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

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

Quote

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

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

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

Quote

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

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

Quote

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

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

Спасибо.

Link to comment
Share on other sites

<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 by andrew312
Link to comment
Share on other sites

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 не уменьшался, нужно добавить свойство.

Спасибо

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

 Share

×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue. See more about our Guidelines and Privacy Policy