Skip to content

YH-2106: add skeleton for SpecializationDetailPage#1291

Open
IvanIvashov wants to merge 2 commits into
developfrom
feature/YH-2106
Open

YH-2106: add skeleton for SpecializationDetailPage#1291
IvanIvashov wants to merge 2 commits into
developfrom
feature/YH-2106

Conversation

@IvanIvashov

Copy link
Copy Markdown
Collaborator

Что сделано

  • Создан компонент SpecializationDetailPageSkeleton
  • Скелетон повторяет структуру страницы специализации
  • Интегрирован через PageWrapper пропс skeleton

@IvanIvashov IvanIvashov added review the task is currently being reviewed feature New functionality or improvements labels Jun 14, 2026

if (isLoading) {
return skeleton || <Loader />;
return skeleton || <Loader />;

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

[SUGGESTION] - лучше следить, чтобы линтер не менял файлы (новые отступы, пробелы и прочее)

<ButtonSkeleton width={130} />
<ButtonSkeleton width={130} />
</BackHeaderSkeleton>
<div className={style.specializationDetailSkeleton}>

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

[ISSUE] - вместо этого дива нужно добавить скелетон для компонента <SpecializationCard> и польностью повторить его структуру:

<Flex>
  <Flex direction=...>
    <CardSkeleton>
      ...

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Поправил

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New functionality or improvements review the task is currently being reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants