Оцените верстку?

Question

[Лизавета]

Добрый вечер!

Можете оценить верстку, пока что не трогаю JS, тренирую HTML+CSS на простых лендингах. Это первый сайт, сверстанный самостоятельно, не по учебным видео
https://github.com/Creepycreep/practice2
https://creepycreep.github.io/practice2/

Comments

[Kentavr16]

можно было бы запариться и хотя бы разместить страницу на github pages. Вряд ли кто-то будет просто так смотреть код по разным файлам)

[szQocks]

Картинки не оптимизировала, адаптива нет, а так же нет стоило бы изучить какую-то методологию по названиям классов

а так в целом вроде нормально, но конечно каждую мелочь не проверял, посмотрел о том соблюдено ли схлопывание а так же где и как заюзала отступы, например секция our_food , у секции ты задала ей марджины, но я например верстаю так что секции прилегают друг к другу, и если секция должна быть чуть ниже якобы другой секции то я задаю ей паддинги а не марджины - но возможно это спорная ситуация

для "первого проекта" это хорошая верстка

и элементы меню сделай ссылками, к примеру ul > li > a > текст
и contact_items phone - за место марджина ( margin-left: 50px; ) должен быть паддинг + там должна быть ссылка с типом номера ссылка
+ почитай про выпадение отступов, хоть у тебя вроде ничего не выпало но знать следует об этом

[szQocks]

Kentavr16, 10 минут заняла вся проверка примерно, склонировал репо , открыл index.html , пробежался быстренько, но будь у меня настроение среднее или плохое я бы даже клонировать не стал)

[Лизавета]

Kentavr16, оу, я просто не в курсе о такой возможности сервиса, изучу

[Лизавета]

szQocks, спасибо, что уделили время)

[Лизавета]

Kentavr16, вы на самом деле этим комментом помогли мне решить одну из проблем, над которой я голову ломала хд

[Антон Литвиненко]

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