Skip to content

Conversation

@dianakuprava
Copy link
Collaborator

@dianakuprava dianakuprava commented Jul 19, 2025

ПРАВКИ 2:

  1. Заменила interface на утилитарный тип
  2. Реализовала динамическую окраску иконок в зависимости от доступности услуги
  3. Вынесла логику склонения слова "место" в утилиту: функция getSeatText создана в shared/utils, согласно FSD.
    Она возвращает корректное окончание в зависимости от числа: "место" | "места" | "мест".
  4. Использование SeatsLeft перенесла в родительский компонент TariffCard, упростила логику
  5. Создала файл types.ts в shared/ui/icon для переиспользования и упрощения типизации всех SVG-иконок
    содержащий общий интерфейс
  6. Подключение Tailwind пока оставила через @import "tailwindcss" напрямую, без выноса в SCSS — для простоты на текущем этапе
  7. Временное размещение логики на HomePage — сделала для наглядности

Добавлены компоненты TicketCard, TicketHeader, TariffsSection, RouteInfo и др.

Иконки вынесены в shared/ui/icons

Реализованы стили и структура билета

Правки:
Разделение TariffsSection на переиспользуемые компоненты:
-Вынесены карточки тарифов в отдельный компонент TariffCard
Исправление форматирования цен:
-Заменены все toLocaleString('ru-RU') на Intl.NumberFormat('ru-RU')
Оптимизация работы с тенями:
-Вынесена тень в константу shadowClass
-Устранено дублирование кода
Улучшения типизации:
-Явные интерфейсы пропсов для всех компонентов
Оптимизация структуры
-Реализованы barrel-экспорты (index.ts) для каждого компонента
-Оптимизированы импорты (абсолютные пути вместо относительных)
-Соответствие FSD
Добавила слово return перед разметкой

Elvira-Elshina
Elvira-Elshina previously approved these changes Jul 28, 2025
Elvira-Elshina
Elvira-Elshina previously approved these changes Aug 1, 2025
Copy link
Collaborator

@Elvira-Elshina Elvira-Elshina left a comment

Choose a reason for hiding this comment

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

В целом всё супер, но для dev-а лучше всё же добавить билет в какой-нибудь роут, но это Леонид уже скажет

TatianaKuzArt
TatianaKuzArt previously approved these changes Aug 1, 2025
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.

5 participants