Кросс-проверка 95/100 #1
Description
С точки зрения функциональных требований сделано очень аккуратно!
| Критерий | Макс.балл | Факт. балл | Пояснение |
|---|---|---|---|
| Технические требования | |||
| Наличие отдельного репозитория проекта | 1 | 1 | ✔ |
| Использование Vite для сборки | 1 | 1 | ✔ |
| Применение CSS Modules для стилей | 1 | 1 | ✔ |
| Модульная организация кода | 4 | 4 | ✔ |
| Использование TypeScript (доп. баллы) | 2 | 0 | Используется JS |
| Применение Zustand в качестве state-менеджера | 2 | 2 | ✔ |
| Использование Fetch для API-запросов | 1 | 1 | ✔ |
| Реализация модальных окон через Portals | 2 | 2 | ✔ |
| Хранение истории в LocalStorage | 1 | 1 | ✔ |
| Использование react-router-dom для роутинга | 2 | 2 | ✔ |
| Настройка ESLint + Prettier | 1 | 0.5 | Нет prettier |
| Отсутствие неразрешенных сторонних библиотек | 6 | 6 | ✔ |
| Соответствие дизайну | |||
| Главная страница | 12 | 10 | По макету было 8 метрик, а в работе 9. В состоянии когда файл загружен дизайн не соответствует макету в части поля для драгндропа. |
| Хедер с меню | 2 | 2 | ✔ |
| Страница генерации таблицы | 2 | 2 | ✔ |
| Страница истории | 8 | 8 | ✔ |
| Функциональные требования | |||
| Загрузка таблицы через drag&drop или кнопку | 4 | 4 | ✔ |
| Отправка таблицы в систему | |||
| Реализация запроса с постепенным получением данных | 8 | 8 | ✔ |
| Данные накапливаются плавно | 2 | 2 | ✔ |
| Есть обработка ошибок | 2 | 2 | ✔ |
| Отображение индикатора загрузки | 4 | 3 | При загрузке лоадер не крутится. |
| По окончании работы запроса в localStorage складывается запись об истории | 2 | 2 | ✔ |
| Кнопка очистить - пропадают хайлайты и подгруженный файл | 4 | 4 | ✔ |
| Роутинг работает, страницы переключаются | 4 | 4 | ✔ |
| При клике на кнопку отправляется запрос за генерацией и автоматически начинается загрузка файла | 4 | 4 | ✔ |
| Есть обработка ошибок | 2 | 2 | ✔ |
| При загрузке страницы данные из localStorage выгружаются и страница инициализируется со списком | 4 | 4 | ✔ |
| Удаление отдельных записей истории | 2 | 2 | ✔ |
| При клике на пункт списка открывается модальное окно с хайлайтами | 6 | 6 | ✔ |
| Полная очистка истории | 2 | 2 | ✔ |
| Кнопка "сгенерировать больше" редиректит на генерацию | 2 | 2 | ✔ |
| Итого | 100 | 95 |
Дополнительные комментарии:
- В истории отображается кнопка "Очистить все" когда нет записей, обсуждали, что ее надо прятать
- При отображении массивов, у элементов должны быть уникальные ключи, иначе могут быть ошибки при отображении + ошибки в консоли
- Можно сделать вывод ошибки, а не просто "упс", чтобы пользователь понимал, что за проблема возникла и мог осознанно обратиться в поддержку
- Можно работать с отображением курсора, чтобы было интуитивно понятно какие элементы кликабельны