Общий суд - покритикуйте работу, пожалуйста)

[Nibaluiso]

Всем доброго, покритикуйте мою работу http://word-master.hostifree.ru- адаптивная верстка. Интересует оценка качества самой верстки и "красоты" кода. Все делалось по макетам psd, поэтому слегка неадекватные размеры шрифтов - не моя вина

P. S.: сколько примерно времени должно уйти на верстку такой адаптивной страницы у верстальщика среднего уровня (лично у меня ушло около 20 рабочих часов).

[vladmih]

1. Ховеров нету на красных кнопулях!!!!
2. Не понятно зачем стоят куча клеарфиксов, даже там где нет плавающих элементов.
3. http://take.ms/MGRkDтуда же можно указать блочную модель, которую указываешь отдельно, по какой-то причине, или может и причины нет...
4. Заголовок первого уровня на странице должен быть ОДИН.
5. сомнительный класс w100 который указывает 100% ширину блоку, который по умолчанию 100%...
6. http://take.ms/akFoLэто должен быть батон
7. http://take.ms/k5REpвысота автоматоически будет калькулироваться и без этого свойства, в случае если ранее не где не указывалось явно свойство высоты... и ширина не где не будет больше 100%, т.к. наше свойство которое указывает всем этим картинкам быть на вс. ширину контейнера, в итоге я не вижу смыслы и height и max-width...
8. Основной шрифт страницы описывай в самом верхнем теге, body или html, не правильно описывать для каждого места одно и то же...
9. http://take.ms/ogDrIэто должно быть одно меню, не должно дублироваться, отображение, меняется через media queries, на то нам и дан респонсив... Не только для того чтоб меня ширину блоков и их расположение

А в общем молодец, даже заморочился с изменением размеров шрифтов для мобильных девайсов... Я представляю сколько тебе пришлось описывать строк))))
Капай в сторону em, rem единиц, упростишь себе жизнь еще больше.

20 часов это конечно много, у меня бы заняло это часов 6-8... Руку набьешь, и будешь делать быстрее

Много лишних классов в местах где они не нужны http://take.ms/4oCQP

 

Edited December 22, 2015 by vladmih

[Nibaluiso]

1. Ховеров нету на красных кнопулях!!!! 2. Не понятно зачем стоят куча клеарфиксов, даже там где нет плавающих элементов. 3. http://take.ms/MGRkDтудаже можно указать блочную модель, которую указываешь отдельно, по какой-то причине, или может и причины нет... 4. Заголовок первого уровня на странице должен быть ОДИН. 5. сомнительный класс w100 который указывает 100% ширину блоку, который по умолчанию 100%... 6. http://take.ms/akFoLэто должен быть батон 7. http://take.ms/k5REpвысота автоматоически будет калькулироваться и без этого свойства, в случае если ранее не где не указывалось явно свойство высоты... и ширина не где не будет больше 100%, т.к. наше свойство которое указывает всем этим картинкам быть на вс. ширину контейнера, в итоге я не вижу смыслы и height и max-width... 8. Основной шрифт страницы описывай в самом верхнем теге, body или html, не правильно описывать для каждого места одно и то же... 9. http://take.ms/ogDrIэто должно быть одно меню, не должно дублироваться, отображение, меняется через media queries, на то нам и дан респонсив... Не только для того чтоб меня ширину блоков и их расположение А в общем молодец, даже заморочился с изменением размеров шрифтов для мобильных девайсов... Я представляю сколько тебе пришлось описывать строк)))) Капай в сторону em, rem единиц, упростишь себе жизнь еще больше. 20 часов это конечно много, у меня бы заняло это часов 6-8... Руку набьешь, и будешь делать быстрее Много лишних классов в местах где они не нужны http://take.ms/4oCQP

 

Спасибо за детальные пояснения! Версткой занимался всего где-то пару месяцев, потом был длительный перерыв и вот с этой работой прямо буксовать слегка начал, долго по времени все это заняло, сами только шрифты описывал часа 2-3 непрерывно)) Буду набивать руку дальше в общем.

[Alex_]

Стоимость: 305 000 рублей - Срок оформления 1 день

 не хило они там живут