slipdem 0 Жалоба Опубликовано: 20 октября, 2016 Добрый день. Просьба посмотреть на верстку и дать оценку/совет/критику по ней. Подскажите, пожалуйста, что в моей верстке плохо, что хорошо а за что вообще по рукам давать надо. Заранее спасибо! Всем хорошего дня Макет тут Верстка тут Поделиться сообщением Ссылка на сообщение Поделиться на других сайтах
klierik 359 Жалоба Опубликовано: 20 октября, 2016 имена классов через "--" я не совсем понял... и баги появляются при узкой ширине http://take.ms/CHh8G #about div { width: 50%; height: 623px; } Лучше через классы всё... .price--plan--row { position: relative; margin-top: 77px; } Ну вот зачем такой класс? и с одним "-" прочитать можно font-size: 22.52px; 23px можно смело писать, ибо пол пикселя не бывает. в целом слабенько, пока что Поделиться сообщением Ссылка на сообщение Поделиться на других сайтах
SelenIT 976 Жалоба Опубликовано: 20 октября, 2016 1 час назад, klierik сказал: ибо пол пикселя не бывает. Бывает, но не на всех платформах. 1 Поделиться сообщением Ссылка на сообщение Поделиться на других сайтах
slipdem 0 Жалоба Опубликовано: 20 октября, 2016 Quote имена классов через "--" я не совсем понял... .price--plan--row { Ну вот зачем такой класс? и с одним "-" прочитать можно Хорошо. Учту. А в целом, что Вы скажете о названиях самих классов? Названия не очень ужасные?) Quote и баги появляются при узкой ширине http://take.ms/CHh8G Мда, там таких багов - греблю гати. Буду разбираться. Лучше все через % и без фиксированной высоты? Quote 23px можно смело писать, ибо пол пикселя не бывает. Поставил пол пикселя, потому что так в шаблоне именно в этом месте было. Но спасибо - буду учитывать в будущем. Quote в целом слабенько, пока что А в чем именно слабо? Что бы Вы посоветовали подтянуть и на что обратить внимание? Спасибо. Поделиться сообщением Ссылка на сообщение Поделиться на других сайтах
klierik 359 Жалоба Опубликовано: 21 октября, 2016 А почитай в подписи типичные ошибки... Поделиться сообщением Ссылка на сообщение Поделиться на других сайтах
andrew312 2 Жалоба Опубликовано: 22 октября, 2016 (изменено) <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 для блока с логотипом.Изменено 22 октября, 2016 пользователем andrew312 Поделиться сообщением Ссылка на сообщение Поделиться на других сайтах
andrew312 2 Жалоба Опубликовано: 22 октября, 2016 <section id="bussines"> </section> #bussines { min-width: 1170px; } Чтобы при уменьшении окна браузера фон и размер section не уменьшался, нужно добавить свойство. Поделиться сообщением Ссылка на сообщение Поделиться на других сайтах
slipdem 0 Жалоба Опубликовано: 27 октября, 2016 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 не уменьшался, нужно добавить свойство. Спасибо Поделиться сообщением Ссылка на сообщение Поделиться на других сайтах