Ivan

1. Фавікон, тайтл: Потрібно самостійно додати фавікон і тайтл. Це показує ваше відношення до роботи і до деталей в цілому 0/1
2. Cursor pointer та Hover на кнопках: Немає Cursor pointer 0/1
3. Помилки в консолі: 1/1
4. Відповідність макету:
   - не підключені шрифти
   - стрілочка не перевертається коли список відкритий
   - немає розділення комою у фільтрах Car mileage, і не відображаються слова "From" і "To" при виборі якогось значення у фільтрах
   - немає лейблів у фільтрів
   1/2
5. Розмітка: Присутні зайві теги div, вертикальний розділювач повинен бути реалізований через стилі, а не текстовим символом 1/2

Результат: 3/7

Репозиторій та код

1. Зайві файли в репозиторії: 1/1
2. Опис файлу README.md: 0/1
3. Назви комітів: 2/2
4. Загальний вигляд репозиторію та коду: Присутній закоменчений код + вищезгадані зауваження 1/2

Результат: 4/6

Запит та пагінація

1. LocalStorage: 2/2
2. Mockapi: 1/1
3. Пагінація: Пагінація має бути реалізована на стороні Mockapi 1/2

Результат: 4/5

Логіка

1. Реалізація модального вікна: Не реалізовано закриття по кліку на esc. Модалка приліплена до початку сторінки, тому якщо відкрити її на останніх картках - її взагалі не видно 1/2
2. Кома реалізована по ТЗ: Кома додається завжди після першої цифри, а якщо це буде 2-значне або 5- і більше значне число? Є простий встроєний метод toLocaleString() 0,5/1
3. Компонентний підхід: 2/2
4. Реалізація логіки:
   Не треба прописувати вручну кожну опцію цін - краще просто створити цикл від 10 до якогось ліміту, з кроком 10.
   Якщо жодні оголошення не підпадають під параметри фільтрів, то показуються всі оголошення.
   Не можливо відфільтрувати по одному або двох параметрах.
   1/3
5. Redux: Зайвий санк getCarsByID 2/3
6. Маршрутизація: При переході за неіснуючим посиланням, видає 404 помилку 1/2
7. dropdown із марками автомобіля: Не можливо відфільтрувати по бренду Land Rover 1,5/2

Результат: 9/15

Додаткове завдання

Реалізація додаткового завдання: Не можливо відфільтрувати по ціні більшій ніж 150$. Немає перевірки що значення "From" менше за "To". Значення в інпутах пробігу відображаються як плейсхолдери і при вводі треба завжди стирати дефолтні значення - це не зручно 3/5

Резюме

В цілому робота виконана не погано. Але є деякі моменти, на які бажано звернути увагу.

Не реалізовано закриття модалки по кліку на esc - це загальноприйнятий функціонал. Модалка приліплена до початку сторінки, тому якщо відкрити її на останніх картках - її взагалі не видно.

Кома додається завжди після першої цифри, а якщо це буде 2-значне або 5- і більше значне число? Є простий встроєний метод toLocaleString().

Є невеличкі зауваження до розмітки і стилізації, а саме: не підключені шрифти, стрілочка не перевертається коли список відкритий, немає розділення комою у фільтрах Car mileage, і не відображаються слова "From" і "To" при виборі якогось значення у фільтрах, немає лейблів у фільтрів, присутні зайві теги div, вертикальний розділювач повинен бути реалізований через стилі, а не текстовим символом.

Пагінація має бути реалізована на стороні Mockapi.

Також є вищеописані зауваження до фільтрації, логіки реалізації та Redux.

На жаль, при оновленні сторінки каталогу або улюблених оголошень, або при переході за неіснуючим посиланням - видає 404 помилку. У реальному житті замовник не перевіряв би таку роботу взагалі. За цю помилку я змушений відняти 1 бал від підсумкової оцінки.

Загальний результат: 23/38

В цілому, оцінка 5/10