Review #3 (Home)

[adadychanka]

Свойство margin-top не применяется к элементу, нужно увеличить специфичность селектора, чтобы оно работало

https://github.com/artyom-voitas/AV-Store/blob/b3b82b15f4e9f1beee6148c6b208e9f749446181/src/styles/_home.scss#L2

---

Код вводящий в заблуждение. В начале модуля вызывается метод logout. Смотря на этот код, я совершаю предположение, что ты при загрузке этого модуля (страницы с модулем) делаешь (зачем-то) выход из приложения. Посмотрев код функции logout я понимаю, что в этой функции просто вешается обработчик на кнопку logout.

• в каждой странице происходит вызов этой функции.

Можно выделать код, касающийся назначения обработчиков событий на кнопки в хедере в отдельный модуль header.js и подключать его ко всем страницам, где показывается хэдер. Это решение более надёжное и уменьшает кол-во дублируещегося кода.

• нужно переназвать функцию. Можно что-то типо assignHandlerToLogoutButton

https://github.com/artyom-voitas/AV-Store/blob/b3b82b15f4e9f1beee6148c6b208e9f749446181/src/js/home.js#L5

---