@DoriDori

Можете покритиковать верстку?

Здравствуйте, оцените верстку, пожалуйста.

Заранее поясню:

CSS написал по принципу важность(как мне кажется)>алфавит
Шрифт не как в макете потому что не хотелось искать его и пиратить
Комментарии делал специально и интересует где они лишние, не хватает и т.п.
Нижний фон добавил в body потому что не придумал как его сверстать. Вообще по фонам у меня отдельный вопрос
Адаптив под 360 не стал делать по очевидным причинам

По моему макет кошмарный в техническом плане, ни логики ни структуры адекватной.

Несколько вопросов:

Можно ли таким способом переносить строки заголовков и параграфов? (кусок текста берется в span и в css display block)
Нужно ли ставить tabindex на блоки, а не только ссылки/кнопки/поля? Или для этого пробел со стрелками используется?
Как обычно верстаются фоны? Например тут 3 картинки. Куда их прикреплять? К блокам? К body можно?
Возможно это странное использование h5 или так можно?

Буду благодарен, если кто-то комплексно разберет. Интересует в правильном направлении иду или нет.

Страница
GitHub
zip с файлами
PSD Макет
Макет картинкой
  • Вопрос задан
  • 898 просмотров
Пригласить эксперта
Ответы на вопрос 4
@s0xzwasd
Почему-то называется один из классов features_items, хотя внутри него всего один осмысленный блок, лучше уж тогда features_item.

option_card_two, option_card_one — вообще странные названия для этих блоков.

Вся верстка в футере едет при добавлении ещё хотя бы одного меню.

При добавлении пары элементов в подписки тоже едет верстка.

12pzTUy.png

При добавлении пунктов меню тоже едет верстка.

Ten3mmG.png
Ответ написан
wapster92
@wapster92
Люблю трилогию ME
Нет адаптива, сейчас не 2008. .next-row что это за фигня, в html для этих целей есть br
Ответ написан
vitaly44
@vitaly44
Предприниматель, веб-разработчик, дизайнер
Комплексно — это надо целый академический курс читать)))

Вообще чтобы критиковать — надо много времени на изучение потратить, чтобы это не делать — задав встречный вопрос.

А почему ты считаешь свою верстку хорошей? Если считаешь, что можно доработать — так доработай)))

А ругать графический дизайн верстальщикам — дело опасное)

Вообще вот пару замечаний.

header надо сделать и в него уже nav поместить, и без дополнительных обёрток, там же просто ссылки, ну ок — ссылку можно обернуть nav_item ))

Ну и далее чихорда какая-то со структурой страницы, для лендинга раздел это section, шапка это header, навигация по разделам это nav и т.д.

main содержит section… skyeer.artbayard.ru для примера можно посмотреть здесь, хотя это и не идеал, но всё же)
Ответ написан
@DLeo13
Почитай БЭМ
Ответ написан
Ваш ответ на вопрос

Войдите, чтобы написать ответ

Войти через центр авторизации
Похожие вопросы
19 сент. 2019, в 23:58
3000 руб./за проект
19 сент. 2019, в 21:14
20000 руб./за проект
19 сент. 2019, в 19:54
6400 руб./за проект