Здесь собираются общие советы, без привязки к конкретной теме.
☝️ - Старайтесь поменьше копипастить код откуда бы то ни было. По крайней мере перечитывайте то, что копипастите. Очень распространенный пример из практики - люди копипастят концепт с CodePen, а потом выясняется, что можно половину кода из него выбросить без потери функционала, да еще и с увеличением производительности. Это не говоря уже о том, что просто понимать, что происходит в проекте, полезно.
☝️ - Закомментированный код лучше убирать. Если что - он останется в истории коммитов, но в актуальной версии проекта он только отвлекает.
☝️ - Такого рода длинные строки можно разбивать на несколько, и форматировать так, чтобы одинаковые сущности были ровно друг над другом. В большинстве случаев так будет проще бегло читать код. Не во всех проектах людьми приветствуются "лишние" пробелы, там есть разные аргументы по этому поводу, но если вы вообще никогда не пробовали так делать - то можно попробовать в этом проекте.
☝️ - Обратите внимание на множественные предупреждения от линтера. Имеет смысл все поправить, чтобы потом в этом хаосе не пропустить что-то действительно важное. Если вы не видите здесь никаких ошибок - проверьте, что линтер в вашей IDE работает и подхватывает конфиги из этого проекта. Может что-то отвалилось.
☝️ - Довольно много ошибок. Кажется, что вы забыли посмотреть видео концепт, в котором поясняется, как именно должны выглядеть те или иные анимации на странице. Посмотрите его внимательно и постарайтесь повторить логику происходящего в нем.
☝️ - Вы движетесь в правильном направлении, но нужно быть внимательнее к деталям. В анимировании это очень важно. И лучше сразу привыкнуть к тому, что все движения в анимации должны быть сделаны в полном объеме. Если вы немного не попали по времени или кривую Безье не ту взяли - это можно оперативно поправить, а вот недостающие состояния элементов, несоответствие форм элементов или не те принципы движения элементов оставляют задачу недоделанной. Часто такие моменты приводят к ситуациям, когда к релизу приходится все переверстывать и по сути делать работу второй раз. Обычно гораздо проще и спокойнее сразу делать то, что ожидается.
☝️ - Такие лендинги часто являются временным дополнением к основному сайту под какую-то рекламную кампанию, и, соответственно, хостятся где попало и как попало. Корень лендинга часто будет не совпадать с корнем сервера, как и в нашем случае с учебным проектом. Так что нужно следить за путями к файлам. Идея использовать абсолютные пути к чему-либо - опасная, равно как и длинные отсылки к родительским директориям вида ../../../../../ - они могут неявно сокращаться в одном окружении, и не сокращаться в другом. Это стоит иметь в виду на будущее. Всегда надежнее сразу проверять пути к файлам, и делать так, чтобы такой сайт работал сам в себе, без завязки на окружение, на настройки какого-то конкретного сервера.
☝️ - Обычно проще работать, когда одна и та же задача решается всегда одним и тем же образом. Когда у нас появляется много разных решений для одной и той же задачи - становится сложно за ними следить. Со временем они начинают рассинхронизироваться все больше, начинаются постоянные запросы на пофиксить тут, пофиксить там, сроки растут, все недовольны. Бардак. Здесь я заметил, что что-то продублировалось, но если бы это был большой проект - скорее всего никто бы не заметил, и все бы попало в продакшен. А потом еще, и еще, и еще. А потом все говорят "легаси, лапша, надо переписать". В учебном проекте можно и нужно экспериментировать, но в будущем - будьте осторожны с такими дублированиями функционала. Они могут потом аукнуться.