Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

skeletonNewsBlock #394

Merged
merged 32 commits into from
Jun 15, 2024
Merged

skeletonNewsBlock #394

merged 32 commits into from
Jun 15, 2024

Conversation

Kseniya554
Copy link
Collaborator

@Kseniya554 Kseniya554 commented May 20, 2024

Ссылка на задачу #309
Пока сделала новостной блок. Правильно ли получилось или не совсем то?
И у меня почему-то изначально в файле NewsBlock.tsx выскакивала ошибка на NewsBlock.

"Тип "() => false | JSX.Element" не может быть назначен для типа "FC".
Тип "false | Element" не может быть назначен для типа "ReactElement<any, any> | null".
Тип "boolean" не может быть назначен для типа "ReactElement<any, any>".ts(2322)
const NewsBlock: FC"

@Kseniya554 Kseniya554 requested a review from aimenin May 20, 2024 20:51
@Kseniya554 Kseniya554 self-assigned this May 20, 2024
Copy link
Collaborator

@aimenin aimenin left a comment

Choose a reason for hiding this comment

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

По поводу ошибки, не могу сказать, что не так, потом посмотрю в проекте.
Скелетон реализован правильно.
Давай подождем pr Александра, подтянем main себе в ветку и можно будет мержить

AlexanderMorugin and others added 22 commits May 21, 2024 19:04
…login_form_adaptive_new

enhancement_377_login_form_adaptive_new
Заменил моковыве данные в корзине в шапке на реальные данные из слайс…
…favorite_page

Enhancement 381 favorite page
…create-Register-Page

#360-enhancement-create-Create-Account-Page
…adaptive_main_page

enhancement_350_adaptive_main_page
@aimenin aimenin self-requested a review June 3, 2024 07:31
Copy link
Collaborator

@aimenin aimenin left a comment

Choose a reason for hiding this comment

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

Скелетон в историях сжимается.
Проверь его реализицию. После этого давай смержим этот pr и дальнейшие изменения будем делать в следующем

@aimenin aimenin self-requested a review June 10, 2024 04:46
Copy link
Collaborator

@aimenin aimenin left a comment

Choose a reason for hiding this comment

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

Размер скелетона в историях не совпадает с размером реального компонента

Copy link
Collaborator

@aimenin aimenin left a comment

Choose a reason for hiding this comment

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

Скелетон должен повторять полностью элемент, который он замещает. В историях в скелетоне почему то появляются еще каких то два поля под картинкой.

@aimenin aimenin self-requested a review June 14, 2024 04:17
Copy link
Collaborator

@aimenin aimenin left a comment

Choose a reason for hiding this comment

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

Вот теперь все правильно

@Kseniya554 Kseniya554 merged commit cc01c0c into master Jun 15, 2024
1 check passed
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