Sprint 1 #33
Sprint 1 #33
Conversation
There was a problem hiding this comment.
Здравствуйте. (Нужно развернуть общий комментарий ↓)
Работа проделана огромная
Что сделано отлично:
Readmeхорошо оформлен- Проект задеплоен и отлично работает
- Отлично, что не забыли про
.gitignore - Отлично, что ограничиваете версию
node - Хорошая структура папок и файлов
- Отлично, что не забываете про
altв тегахimg
Что нужно исправить:
- По семантике нужно помещать весь главный контент страницы в тег
main. - Нужно сверстать еще страницу
редактирования данных профиля, она должна быть с формой и инпутами, у инпутов должны быть атрибутыnameпо заданию
Что можно улучшить:
- В проекте в некоторых файлах есть ошибка
EOF. Дляgitважно наличие пустой строки в конце файла. Подробнее тут: https://stackoverflow.com/questions/5813311/whats-the-significance-of-the-no-newline-at-end-of-file-log Можно в настройки IDE добавитьinsert_final_newline = true - все цвета лучше вынести в
css-переменные и использовать их, чтобы можно было мгновенно понимать, что это за цвет, и можно было бы подкорректировать его в одном месте и не искать дублирования во всем коде
Исправьте, пожалуйста, недочеты и работа будет принята. Пожалуйста, проверьте работоспособность проекта и наличие возможных ошибок в консоли браузера (кнопка F12) перед отправкой на ревью.
Напоминаю, что работа может быть принята только после исправления всех критических замечаний Нужно исправить.
Комментарии Можно лучше не обязательны к исправлению прямо сейчас, это рекомендации
Удачного рефакторинга кода.
src/404.html
Outdated
| @@ -0,0 +1,2 @@ | |||
| <template data-type="pug" data-src="./pages/404.pug"></template> | |||
| <script type="module" src="style.js"></script> | |||
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/components/form/form.scss
Outdated
| width: 400px; | ||
| max-width: 100%; | ||
| border-radius: var(--b-radius); | ||
| background-color: #f5f5f5; |
There was a problem hiding this comment.
Можно лучше
все цвета лучше вынести в css-переменные и использовать их, чтобы можно было мгновенно понимать, что это за цвет, и можно было бы подкорректировать его в одном месте и не искать дублирования во всем коде
src/layout/main.pug
Outdated
|
|
||
| body | ||
| .page__content | ||
| block content |
There was a problem hiding this comment.
Нужно исправить
По семантике нужно помещать весь главный контент страницы в тег main.
Чтобы сделать сайт более доступным для пользователя, нужно стараться придерживаться семантической верстки.
Несколько полезных статей на эту тему:
• Что такое семантическая вёрстка и зачем она нужна
• Семантика для циников
src/pages/user-settings.pug
Outdated
| .user-settings__f | ||
| .user-settings__avatar | ||
| h2.user-settings__name User name | ||
|
|
There was a problem hiding this comment.
Нужно исправить
- Нужно сверстать еще страницу
редактирования данных профиля, она должна быть с формой и инпутами, у инпутов должны быть атрибутыnameпо заданию
|
Привет. Добавил commit с исправлениями.
|
…atar error, added BASE_URL const etc...
Chickie brickie