YH-2029: update skeleton for CollectionPage#1277
Conversation
IrinaYarutkina
added
bug
| "WebFetch(domain:github.com)" | ||
| ] | ||
| } | ||
| "permissions": { |
There was a problem hiding this comment.
Тут форматер отработал, такие пустые изменения(не связанные с задачей) лучше не заносить в коммит
There was a problem hiding this comment.
[SUGGESTION] - Я бы даже сказал, что лучше удалить эту папку. Видимо в какой то задаче ктото занес
|
|
||
| import styles from './CollectionPage.module.css'; | ||
|
|
||
| export const CollectionPageSkeleton = () => { |
There was a problem hiding this comment.
[ISSUE] - Сейчас скелетон не отрабатывает корректно, так как в CollectionPage стоит проверка на наличие коллекции. Ее нужно убрать, так как до получения коллекции показывается заглушка, которой не должно быть
ПОсле того как уберешь это условие нао будет переместить все что передается в пропс content компонента PageWrapper в отдельный компонент CollectionPageContent. И соответственно скелетон также расположить
| ))} | ||
| <Flex direction="column" gap="8"> | ||
| <TextSkeleton variant="body3" width={62} /> | ||
| <Skeleton width={70} height={30} /> |
There was a problem hiding this comment.
[ISSUE] - Тут нужно использовать StatusChipSkeleton
| <TextSkeleton variant="body3" width={62} /> | ||
| <Skeleton width={70} height={30} /> | ||
| </Flex> | ||
| <BaseFilterSectionSkeleton length={2} /> |
There was a problem hiding this comment.
[ISSUE] - Следующие ниже 4 элемента также должны быть BaseFilterSectionSkeleton но с length 1
| <TextSkeleton variant="body3" width={137} /> | ||
| <Skeleton width={41} height={39} /> | ||
| </Flex> | ||
| <Skeleton width={270} height={60} /> |
There was a problem hiding this comment.
[ISSUE] - Здесь должен быть AuthorInfoSkeleton
|
|
||
| import styles from './CollectionBody.module.css'; | ||
|
|
||
| export const CollectionBodySkeleton = () => { |
There was a problem hiding this comment.
[ISSUE] - Здесь совершенно не соответствует структура основному компоненту
Вместо Card -> CardSkeleton и чтобы не отображался заголовок в виде текста, а в виде скелетона
Внутри мапинга использовать PreviewQuestionsItemSkeleton, но проверить чтобы корректно отображался
PerelomaDenis
added
comments
3 participants
YH-2029
Также поправлены дочерние компоненты:
-AdditionalInfoSkeleton