Признаки низкого качества вёрстки

То, что практикуют говноверстальщики.

• Невалидный (X)HTML. Например, div внутри a.
• Обилие grid layout везде, где надо и где не надо.
• В целом обилие суперновых фич и низкий уровень совместимости с различными браузерами — IE-то, к счастью сдох, но на этом всё не закончилось и, например, браузер заднеприводных (Safari) тоже иногда ведёт себя не так, как хром и ФФ.
• Вёрстка таблиц div-ами. С хардкодом ширины и высоты ячеек.
• Неустойчивость к контенту (хардкод в целом). Например, съезжание элементов при добавлении пары слов, отсутствие заглушек для фото…
• Использование JS. Вёрстка должна быть в рамках HTML+CSS, JS — это удел разработчика, верстальщик его должен применять только в самых крайних случаях.
  • Некоторые дристальщики до сих пор не могут оторваться от сиси jQuery.
  • Некоторые используют swiper вместо вёрстки.
• Обилие :nth-child, означающее жёсткие завязки на число («только 6 картинок») или позиции элементов («первый элемент красный»).
• Использование SASS. Не выпендривайтесь, пишите голый CSS. Профита от SASS почти ноль, автопрефиксер потом припилить — разработчик справится, а вот переделать SASS например в CSS Modules — проблемка.
• Новое слово в жанре: использование svg-спрайтов. Откройте для себя уже иконочные шрифты.
• Отсутствие «общих» элементов. Например, заданного стандартного размера шрифта на страницу, стилей текста, ссылок, более-менее стандартизированных и переиспользуемых элементов управления, форм и т. п.

Резюмировать можно так: стройная система остаётся стройной в любом положении. Так и вёрстка должна иметь устойчивость и сохранять адекватный внешний вид, когда разработчик её меняет в пределах разумного или сочиняет новые странички на основе существующих.