- Code Quality
- Performance
- Functionality
- User Experience
- Testing
- Documentation
Плюси: Використання семантичних елементів HTML: header, section, footer, які покращують читабельність. Чіткий розподіл розділів і відповідне використання коментарів.
Мінуси: використання вбудованих стилів, краще виносити в окремий css файл. Багато великих секцій, що може вплинути на підтримку та читабельність. Розбиття їх на більш дрібні багаторазові компоненти може покращити це.
Плюси: Чітке використання jQuery для роботи з DOM і обробки подій. Мінуси: Використання console.log
Аналіз за допомогою Lighthouse показує високі показники First Contentful Paint і Largest Contentful Paint
Працює добре. Не знаю, що додати.
Трохи задовго грузиться "лоадер". Не значні стильові похибки в мобільному режимі.
Т.я. це невеличкий лендінг, то наявність тестів та документації не потрібні.
Важко оцінити, т.я. я працюю фронт-енд девелопером. Єдине, що можу сказати (але це відомо було заздалегідь) - великий мінус це відсутніть негайного оновлення кількості місць для того чи іншого пакету.
В цілому оцінка проекту 9 із 10.
Сподіваюся ви вже ніколи не будете таким добрим у виставленні оцінок )))
Коли перераховуєте мінуси завжди раджу описувати "а що з цим робити". Це дозволить одразу надати цей документ умовному джуну, най править.
Взагалі дуже круто що згадали про лайтхауз. А про те чи потрібні тести поговоримо вже скоро :)