YH-1564: add skeleton for analytics pages#1240
Conversation
Gnarkill33
added
review
| } | ||
|
|
||
| export const AnalyticPageTemplateSkeleton = ({ | ||
| rowsCount = 10, |
There was a problem hiding this comment.
[SUGGESTION] - думаю что rowsCount не понадобиться, так как пускай всегда будет 10
There was a problem hiding this comment.
Убрала rowsCount
| fieldsCount={fieldsCount} | ||
| /> | ||
| ) : ( | ||
| <table> |
There was a problem hiding this comment.
[ISSUE] - Укажи тут лучше TableSkeleton указав нужное количество столбцов. Единственное надо будет в TableSkeleton столбец с actions под условие поставить под новый пропс - hasAction = true. И в твоем случае его пропсами указать hasAction false и hasSelectors = false
There was a problem hiding this comment.
Заменила на TableSkeleton, добавила условие
| }; | ||
|
|
||
| if (isLoading) { | ||
| return skeleton || <Loader />; |
There was a problem hiding this comment.
[ISSUE] - Нельзя такое убирать не согласовав, так как компонент много где используется
Добавь в SkeletonGenerator который в MainLayout новые case под каждую страницу и добавь такие же скелетоны как были добавлены дял PageWrapper
И тогдла это изменение не понадобится
There was a problem hiding this comment.
Вернула лоудер, добавила скелетоны
PerelomaDenis
added
comments
…te and template mobile list skeletons
…t in analytic page skeleton to 3
…ayout skeleton generator
Gnarkill33
added
review
2 participants
Тикет
YH-1564
Описание изменений