BOX-60 BOX-26 User page and card view page #27
Conversation
BOX-60 Страница пользователя
|
sergeysova
changed the title
BOX-26 Просмотр карточки
Card (v.0) - Cardbox Main (Figma) У каждой карточки должна быть прямая ссылка, которая не меняется при редактировании |
azinit
left a comment
azinit
left a comment
There was a problem hiding this comment.
В целом - оч добротно, но некоторые моменты бы поправил всеж
Возможно со второй итерации еще что вылезет, но в целом есть что улучшить еще)
|
Плюс я бы более осмысленное описание изменений делал И тебе проще рефлексировать "А не задел ли я что-то лишнее по задаче?" и ревьюверу проще понять, что тут происходит
|
# Conflicts: # yarn.lock
azinit
left a comment
azinit
left a comment
There was a problem hiding this comment.
Есть что улучшить офк, но я бы лично поправил это позж, а пока залил бы как есть))
А то сколько времени уже тянем
|
|
||
| return ( | ||
| <PaperContainerStyled data-type={type}> | ||
| {/*<Container>*/} |
There was a problem hiding this comment.
Твоя любовь к комментарием все не отпускает тебя, да?)
| import { avatarUri } from '../../../shared/constants'; | ||
| import { paths } from '../../../pages/paths'; |
There was a problem hiding this comment.
Некритично, но я бы уже начинал абсолютные пути юзать
| import { avatarUri } from '../../../shared/constants'; | |
| import { paths } from '../../../pages/paths'; | |
| import { avatarUri } from '@box/shared/constants'; | |
| import { paths } from '@box/pages/paths'; |
| export const UserPage = () => { | ||
| useStart(model.pageLoaded); | ||
| const cards = useStore(cardModel.$cards); | ||
| const reversedCards = [...cards].reverse(); |
There was a problem hiding this comment.
У тебя эта операция каждый рендер будет происходить
Некритично, но я бы в самой модели смаппил
$cards.map(...) (насколько помню так в эффекторе делается)
| @@ -0,0 +1,2 @@ | |||
| export const avatarUri = | |||
There was a problem hiding this comment.
Потом из вьювера сможешь брать)
| const onClick = React.useCallback(() => { | ||
| setSelectedTab(index); | ||
| }, [setSelectedTab, index]); |
There was a problem hiding this comment.
|
Будет пофикшено по: |
feat/user-profile