Skip to content

Conversation

@eugenekh81
Copy link

Copy link

@varseniuk varseniuk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

В цілому - хороша робота. Але є певні нюанси:

  • шрифт не відповідає дизайну
  • клік по лого дає 404 помилку
  • бургер меню, на мою думку, варто рендерити при меншій ширині. Бо зараз забагато вільного простору навколо пунктів

@eugenekh81
Copy link
Author

принял, буду добаратывать. Хотел спросить, как избавиться от ошибки Test / build (12.x) (pull_request) Failing after 46s — build (12.x), я могу как-то на это со своей стороны повлиять (что-то изменить в проекте), и если да, то как это сделать ?

Спасибо )))

@eugenekh81
Copy link
Author

eugenekh81 commented Jan 5, 2022

  • шрифт не відповідає дизайну
  • клік по лого дає 404 помилку
  • бургер меню, на мою думку, варто рендерити при меншій ширині. Бо зараз забагато вільного простору навколо пунктів

Список исправлений:

  1. Проверил шрифты, привел в соответствие с макетом, если где-то что-то упустил, прошу указать где, возможно не все увидел и исправил
  2. вставил ссылку на страничку, протестировал клик на лого вверху странички - переход на главную страницу (вверх страницы), ошибку 404 не выдает
  3. Добавил отображение номера телефона при меньшей ширине + гамбургер меню, т.к. все меню не помещается горизонтально

Спасибо за ревью :)

@eugenekh81 eugenekh81 requested a review from varseniuk January 5, 2022 17:01
Copy link

@danylo-onishchuk danylo-onishchuk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good job!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants