Sprint 1 #56
Sprint 1 #56
Conversation
There was a problem hiding this comment.
Здравствуйте. (Нужно развернуть общий комментарий ↓)
Работа проделана огромная
Что сделано отлично:
Readmeхорошо оформлен- Отлично, что не забыли про
.gitignore - Хорошая структура папок и файлов
- Отлично, что цвета вынесены в
css-константы - Отлично, что не забываете про
altв тегахimg
Что нужно исправить:
- При переходе по ссылке на нетлифай страница недоступна, скрин https://skr.sh/sOGUm1o2IfY
- нужно ограничить версию
nodeвpackage.jsonпримерно так:"engines" : { "node" : ">=12" }, чтобы она была не ниже12версии. Это есть в чек-листе - По заданию 1го спринта нужно давать инпутам конкретные атрибуты
name, чтобы в 3м спринте не менять ничего при подключении кApi. Скрин из задания https://disk.yandex.ru/i/lSPqeGrAa20ayQ - Инпуты и кнопку сабмита по семантике нужно обернуть в тег
form, так как это форма - По семантике нужно помещать весь главный контент страницы в тег
main.
Что можно улучшить:
- В проекте в некоторых файлах есть ошибка
EOF. Дляgitважно наличие пустой строки в конце файла. Подробнее тут: https://stackoverflow.com/questions/5813311/whats-the-significance-of-the-no-newline-at-end-of-file-log Можно в настройки IDE добавитьinsert_final_newline = true
Исправьте, пожалуйста, недочеты и работа будет принята. Пожалуйста, проверьте работоспособность проекта и наличие возможных ошибок в консоли браузера (кнопка F12) перед отправкой на ревью.
Напоминаю, что работа может быть принята только после исправления всех критических замечаний Нужно исправить.
Комментарии Можно лучше не обязательны к исправлению прямо сейчас, это рекомендации
Удачного рефакторинга кода.
package.json
Outdated
| "dependencies": { | ||
| "handlebars": "^4.7.8" | ||
| } | ||
| } |
There was a problem hiding this comment.
Нужно исправить
нужно ограничить версию node в package.json примерно так: "engines" : { "node" : ">=12" }, чтобы она была не ниже 12 версии. Это есть в чек-листе
| </div> | ||
| </div> | ||
|
|
||
| </div> |
There was a problem hiding this comment.
Можно лучше
В проекте в некоторых файлах есть ошибка EOF. Для git важно наличие пустой строки в конце файла. Подробнее тут: https://stackoverflow.com/questions/5813311/whats-the-significance-of-the-no-newline-at-end-of-file-log Можно в настройки IDE добавить insert_final_newline = true
src/pages/SignIn/signIn.hbs
Outdated
| <h1 class="signIn__title">Регистрация</h1> | ||
| <div class="signIn__inputs"> | ||
| <label class="signIn__label" for="signIn-email">Почта</label> | ||
| <input id="signIn-email" type="email" class="signIn__input" /> |
There was a problem hiding this comment.
Нужно исправить
По заданию 1го спринта нужно давать инпутам конкретные атрибуты name, чтобы в 3м спринте не менять ничего при подключении к Api. Скрин из задания https://disk.yandex.ru/i/lSPqeGrAa20ayQ
| <div class="profile__sidebar"> | ||
| <button class="profile__btn"><a href="/" class="nav__link"></a></button> | ||
| </div> | ||
| <div class="profile__info"> |
There was a problem hiding this comment.
Нужно исправить
Инпуты и кнопку сабмита по семантике нужно обернуть в тег form, так как это форма
Чтобы сделать сайт более доступным для пользователя, нужно стараться придерживаться семантической верстки.
| <title>Vite App</title> | ||
| </head> | ||
| <body> | ||
| <div id="app"></div> |
There was a problem hiding this comment.
Нужно исправить
По семантике нужно помещать весь главный контент страницы в тег main.
| class="profile__label" | ||
| for="profile__input-email" | ||
| >Почта</label> | ||
| <input |
There was a problem hiding this comment.
Нужно исправить
По заданию 1го спринта нужно давать инпутам конкретные атрибуты name, чтобы в 3м спринте не менять ничего при подключении к Api. Скрин из задания https://disk.yandex.ru/i/lSPqeGrAa20ayQ
No description provided.