-
Notifications
You must be signed in to change notification settings - Fork 0
Conversation
…ions, wrap in ui/backgroung navigation, hero and about sections in index.page
…, check layout, check typography in process section
…ider sections, change layout, font color in slider section
Скоуп Описание Мои действия
Вопросы Какой механизм анимации? |
BREAKING CHANGE: incorrect parallax props
Давай пока начнем с очевидных фиксов:
|
…, parallax only in desktop
…ess css props, rework button component
landing/fragments/landing-navigation/src/navigation.component.tsx
Outdated
Show resolved
Hide resolved
…ption in process component
landing/fragments/landing-navigation/src/navigation.component.tsx
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Странно, у меня кроме адаптивности карточек всё хорошо. Смотрел в хроме и мозиле, точно ветка третьего степа.
Details
Screencast.from.15.08.2023.19.59.59.webmScreencast.from.15.08.2023.20.00.39.webm |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Необходимы правки.
|
||
<Box flexDirection={['column', 'row']}> | ||
<Column order={[3, 0]} flexBasis={[335, 1160]} flexGrow='1'> | ||
<Card |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Давай все карточки перепишем через .map
. Создаешь список/объект из данных карточек, и его мапишь с return (<Card ... />)
. Иначе ты под каждую карточку повторяешь код.
<Item question={intl.formatMessage({ id: 'faqQuestion' })} divider={1} /> | ||
<Item question={intl.formatMessage({ id: 'faqQuestion' })} divider={1} /> | ||
</Column> | ||
<Item question={intl.formatMessage({ id: 'faq.question' })} /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Давай сделаем отдельно выше список из 6 элементов, его размапим и вернем <Item ... />
.
tsconfig.json
Outdated
@@ -34,5 +34,5 @@ | |||
} | |||
] | |||
}, | |||
"include": ["auth/**/*", "landing/**/**/*", "ui/**/**/*"] | |||
"include": ["auth/**/*", "landing/**/**/*", "ui/**/**/**/**/*"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
А зачем такая вложенность в паттерне?
import { CardsMaterials } from './cards' | ||
import { CardsSwiper } from './cards' | ||
import { CardsLearning } from './cards' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
я, как и ранее - не рекомендовал бы косметические отличия выдавать за новую сущность, на 7 этапе будет бобо)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
У меня тут три разных компановки карточек.
Т.е. сущность одна - карточка, просто в одном случае показывает карточки с категорией учебный материал
, в другом карточки собираются в свайпер, в третьем карточки с категорией обучение
или мини-курс
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
а должна быть одна карточка, которая где-либо расширяется и переиспользуется
<Box display='inline' maxWidth={[335, 710]}> | ||
<Text | ||
color='text.black' | ||
display='inline' | ||
color='text.primary' | ||
fontSize={['small', 'ordinary']} | ||
fontWeight='normal' | ||
lineHeight={['normal', 'regular']} | ||
> | ||
<FormattedMessage id='coursesSubtitleAtlantisPowerBroker' /> | ||
<FormattedMessage id='courses.subtitle.power-broker' /> | ||
</Text> | ||
|
||
<Space count={2} /> | ||
|
||
<Text | ||
display='inline' | ||
color='text.accent' | ||
fontSize={['small', 'ordinary']} | ||
lineHeight={['normal', 'regular']} | ||
> | ||
<FormattedMessage id='courses.subtitle.atlantis' /> | ||
</Text> | ||
|
||
<Space count={1} /> | ||
|
||
<Text | ||
display='inline' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
зачем тут везде инлайны выставленны?
id: 0, | ||
question: 'Подойдёт ли мне профессия?', | ||
}, | ||
{ | ||
id: 1, | ||
question: 'Подойдёт ли мне профессия?', | ||
}, | ||
{ | ||
id: 2, | ||
question: 'Подойдёт ли мне профессия?', | ||
}, | ||
{ | ||
id: 3, | ||
question: 'Подойдёт ли мне профессия?', | ||
}, | ||
{ | ||
id: 4, | ||
question: 'Подойдёт ли мне профессия?', | ||
}, | ||
{ | ||
id: 5, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
к чему копипаста? можно ведь прогнать через цикл один объект
Close #12