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

Question

[DoriDori]

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

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

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

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

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

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

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

Страница
GitHub
zip с файлами
PSD Макет
Макет картинкой

Answer 1

[Даниил Маслов]

Почему-то называется один из классов features_items, хотя внутри него всего один осмысленный блок, лучше уж тогда features_item.

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

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

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



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

Comments

[Brad9aga]

я те по секрету скажу, она и без добавления уже съехала

[Даниил Маслов]

Brad9aga, я не тестил на разрешениях, так как адаптива все равно нету

[Brad9aga]

s0xzwasd, я просто открыл ссылку, горизонтальный скролл уже по умолчанию присутствует

[Brad9aga]

s0xzwasd, такое ощущение, что верстают даже не проверяя свои действия

Answer 2

[Виталий Кузнецов]

Комплексно — это надо целый академический курс читать)))

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

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

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

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

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

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

main содержит section… skyeer.artbayard.ru для примера можно посмотреть здесь, хотя это и не идеал, но всё же)

Answer 3

[Leo]

Почитай БЭМ

Comments

[Vaultboy84]

Зачем ему бэм? Тут до бэма еще как до китая. Надо освоить адаптив.