Как проверить качество верстки от верстальщика?

Question

[vladislavik95]

Сегодня верстальщик прислал файлы сайта. Я не программист и не знаю как проверить качество верстки. Можете дать свою оценку. Вот одна из страниц front-end.topden.net/coupon/kupon.html
и все страницы -- front-end.topden.net/coupon

Хотя если смотреть по дизайну то визуально дизайн отличается многими вещами. Но буду каждый день его улучшать еще и надеюсь получится вишенька как говорят.

Comments

[Максим Тимофеев]

а что css делает перед </body>?
и js надо бы объединить и сжать

[vladislavik95]

Максим Тимофеев, а как объединить эти js?)

[Максим Тимофеев]

vladislavik95, много способов есть, я юзаю
https://www.npmjs.com/package/gulp-concat для объединения
https://www.npmjs.com/package/gulp-uglify для сжатия

[Дархан]

Не забудьте про теги для SEO.

[vladislavik95]

Дархан Камалиев, ну это уже потом, а как кстати дизайн?) и пользовались ли вы таким сервисом когда то?)

[Дархан]

vladislavik95, я конечно не дизайнер,но в целом выглядит неплохо(субьективно).
Не пользовался,но наслышан(chocolife(не реклама)).

[vladislavik95]

Дархан Камалиев, хорошо, спасибо. Ещё буду тестировать цвета.. И ещё улучшать юзабилити....

[snap44]

Максим Тимофеев,

а что css делает перед </body>?

Так разрешено в html5.2

[Максим Тимофеев]

snap44,

Так разрешено в html5.2

Странный коммент. Разрешено и так надо - разные вещи. Я разве говорил, что спецификация это запрещает? Спецификация не запрещает вставить перед 29 скриптов размером в 20 мб каждый, но это не значит что так надо делать.
Закон РФ (а возможно и других стран) не запрещает Вам обмазаться кетчупом и бегать по своей квартире с привязанной к голове скотчем батарейкой формата AAA. Но будет ли это разумным? Особенно, когда есть AA батарейки.

Answer 1

[Владимир Проскурин]

А какой ответ вы хотите получить? Все зависит от того, какие ваши требования кроссбраузерности/валидности/бэм/семантичности и.т.д. Прям чего-то ужасного в этой верстке нет. Просто смотрите на качество кода (чистый ли он, или из кучи #block1, #block2, #block3), смотрите на правильность отображения верстки, ничего ли не едет на каких-то экранах, попробуйте где-то добавить больше текста (поедет он или нет), или наоборот, убрать текст или блоки.

Comments

[vladislavik95]

согласен, наверное нужно более конкретно задавать вопросы.

Answer 2

[VoidVolker]

Для проверки качества верстки (да и всего остального тоже) есть такая штука как QA, дизайн и требования к верстке. QA осуществляется тестерами — они проверяют соответствие верстки дизайну и требованиям.

Answer 3

[Александр]

Я глянул, все более или менее нормально, но как таковые сайты для проверки качества кода не существуют. Это как проверять смысловую правильность построения предложений в книге

Comments

[vladislavik95]

Хорошо, спасибо за ответ)