Sprint 1 #2
Sprint 1 #2
Conversation
There was a problem hiding this comment.
Здравствуйте. (Нужно развернуть общий комментарий ↓)
Работа проделана огромная
Что сделано отлично:
- Проект задеплоен и отлично работает
- Отлично, что не забыли про
.gitignore - Отлично, что ограничиваете версию
node - Хорошая структура папок и файлов
- Отлично, что цвета вынесены в
css-константы - Отлично, что не забываете про тег
mainдля главной части страницы
Что нужно исправить:
- В README.md нужно добавить ссылки на нарисованные прототипы
- Проект не запускается локально, скрин https://skr.sh/sItdGOfUnLb
Что можно улучшить:
- В проекте в некоторых файлах есть ошибка
EOF. Дляgitважно наличие пустой строки в конце файла. Подробнее тут: https://stackoverflow.com/questions/5813311/whats-the-significance-of-the-no-newline-at-end-of-file-log Можно в настройки IDE добавитьinsert_final_newline = true - Лучше давать разный
typeдля разных кнопок - Лучше указать на главной странице все ссылки на все страницы проекта (включая 404 и 500), чтобы можно было по ним пробежаться быстро и посмотреть верстку.
- Лучше установить корректный язык страницы
<html lang="en">
Исправьте, пожалуйста, недочеты и работа будет принята. Пожалуйста, проверьте работоспособность проекта и наличие возможных ошибок в консоли браузера (кнопка F12) перед отправкой на ревью.
Напоминаю, что работа может быть принята только после исправления всех критических замечаний Нужно исправить.
Комментарии Можно лучше не обязательны к исправлению прямо сейчас, это рекомендации
Удачного рефакторинга кода.
|
|
||
|  | ||
| Чат в ретро стиле с олдскульными шрифтами картинками, основная идея совместить современные технологии со старыми приемами в веб дизайне | ||
|
|
There was a problem hiding this comment.
Нужно исправить
- В README.md нужно добавить ссылки на нарисованные прототипы
src/components/button/index.pug
Outdated
| @@ -0,0 +1,2 @@ | |||
| mixin button(text, className="") | |||
| button(class=`button ${className}`)= text | |||
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
| "express": "^4.18.2", | ||
| "pug": "^3.0.2" | ||
| } | ||
| } |
There was a problem hiding this comment.
Нужно исправить
- Проект не запускается локально, скрин https://skr.sh/sItdGOfUnLb
src/components/button/index.pug
Outdated
| @@ -0,0 +1,2 @@ | |||
| mixin button(text, className="") | |||
| button(class=`button ${className}`)= text | |||
There was a problem hiding this comment.
Можно лучше
Лучше давать разный type для разных кнопок
Например, если это кнопка сабмита, то у ей лучше дать type = "submit", иначе, не будет работать встроенная логика обработки формы по нажатию на Enter
src/pages/index.pug
Outdated
| link(rel='mask-icon' href='../../static/favicon/safari-pinned-tab.svg' color='#5bbad5') | ||
| script(type='module' src='index.js' defer='') | ||
| title Chat | ||
| main#root |
There was a problem hiding this comment.
Можно лучше
Лучше указать на главной странице все ссылки на все страницы проекта (включая 404 и 500), чтобы можно было по ним пробежаться быстро и посмотреть верстку.
Нужно обернуть список с ссылками в тег nav по семантике, так как это навигация по сайту. Будет nav, потом ul, а потом уже li.
Список ссылок выглядит как список, значит, и отрисовываться должен с помощью ul и li. Это стандартная практика, чтобы улучшить UX (это нужно для парсеров сайтов и для спец возможностей браузера). A внутри li уже тег a будет.
| @@ -0,0 +1,14 @@ | |||
| doctype html | |||
There was a problem hiding this comment.
Можно лучше
Лучше установить корректный язык страницы <html lang="en">
Раз интерфейс проекта англоязычный, то и декларация языка должна ему соответствовать. Некоторые устройства могут постоянно предлагать перевести сайт на другой язык, или ничего не спрашивая, тихо менять страницу. Вот одна из таких историй: https://habr.com/ru/post/650907/
windows doesnt support asterisk naming
|
По поводу того что не запускается локально проект. Возможно проблема не у меня, я поставил винду на другую свою машину, c нуля все установил и запустил успешно проект. Обсуждение аналогичной проблемы: Ссылка на скачивание нужного файла для её решения: |
No description provided.