Sprint 1_3 #51
Sprint 1_3 #51
Conversation
add flatten
There was a problem hiding this comment.
Здравствуйте. (Нужно развернуть общий комментарий ↓)
Посмотрите на гитхабе все комментарии к коду (нужно прокрутить вниз страницу там)
Работа проделана огромная
Что сделано отлично:
Readmeхорошо оформлен- Проект задеплоен и отлично работает
- Отлично, что не забыли про
.gitignore - Отлично, что ограничиваете версию
node - Отлично, что нет
EOFошибок в гите - Хорошая структура папок и файлов
- Отлично, что не забываете про тег
mainдля главной части страницы
Что нужно исправить:
- Нужно сверстать еще страницу
редактирования данных профиля, она должна быть с формой и инпутами, у инпутов должны быть атрибутыnameпо заданию
Что можно улучшить:
- все цвета лучше вынести в
css-переменные и использовать их, чтобы можно было мгновенно понимать, что это за цвет, и можно было бы подкорректировать его в одном месте и не искать дублирования во всем коде - Каждой картинке (тегу
img) нужен информативный атрибутalt(например: имя картинки вставлять), так как при ошибке загрузки картинки на сайте подсветится название этой картинки (которое вalt). Еще это нужно для плохо видящих инезрячихлюдей - для них есть специальные средства, которые распознают текст вaltи проговаривают его. Подробнее тут https://habr.com/ru/post/341810/ - Лучше установить корректный язык страницы
<html lang="ru">
Исправьте, пожалуйста, недочеты и работа будет принята. Пожалуйста, проверьте работоспособность проекта и наличие возможных ошибок в консоли браузера (кнопка F12) перед отправкой на ревью.
Напоминаю, что работа может быть принята только после исправления всех критических замечаний Нужно исправить.
Комментарии Можно лучше не обязательны к исправлению прямо сейчас, это рекомендации
Удачного рефакторинга кода.
| box-sizing: border-box; | ||
| text-decoration: none; | ||
| &:hover { | ||
| color: #726969; |
There was a problem hiding this comment.
Можно лучше
все цвета лучше вынести в css-переменные и использовать их, чтобы можно было мгновенно понимать, что это за цвет, и можно было бы подкорректировать его в одном месте и не искать дублирования во всем коде
src/pages/profile/profile.tmpl.js
Outdated
|
|
||
| <div class="profile_content"> | ||
| <div class="profile_logo"> | ||
| <img src=""> |
There was a problem hiding this comment.
Нужно исправить
Каждой картинке (тегу img) нужен информативный атрибут alt (например: имя картинки вставлять), так как при ошибке загрузки картинки на сайте подсветится название этой картинки (которое в alt). Еще это нужно для плохо видящих и незрячих людей - для них есть специальные средства, которые распознают текст в alt и проговаривают его. Подробнее тут https://habr.com/ru/post/341810/
src/index.html
Outdated
| @@ -0,0 +1,17 @@ | |||
| <!doctype html> | |||
| <html lang="en"> | |||
There was a problem hiding this comment.
Можно лучше
Лучше установить корректный язык страницы <html lang="ru">
Раз интерфейс проекта русскоязычный, то и декларация языка должна ему соответствовать. Некоторые устройства могут постоянно предлагать перевести сайт на другой язык, или ничего не спрашивая, тихо менять страницу. Вот одна из таких историй: https://habr.com/ru/post/650907/
|
|
||
| export const label = ({text, url}) => Handlebars.compile(` | ||
| <button class="button button__blue" onclick="window.location.href ='{{ url }}'">{{ text }}</button>`)({text, url}) | ||
|
|
There was a problem hiding this comment.
Нужно исправить
- Нужно сверстать еще страницу
редактирования данных профиля, она должна быть с формой и инпутами, у инпутов должны быть атрибутыnameпо заданию
add newline at end of files