Skip to content
This repository has been archived by the owner on Nov 29, 2023. It is now read-only.

feat/landing-third-step #13

Merged
merged 44 commits into from
Aug 21, 2023
Merged

feat/landing-third-step #13

merged 44 commits into from
Aug 21, 2023

Conversation

Mirved64
Copy link
Contributor

@Mirved64 Mirved64 commented Aug 9, 2023

Close #12

@Mirved64 Mirved64 added the enhancement New feature or request label Aug 9, 2023
@Mirved64 Mirved64 self-assigned this Aug 9, 2023
@Mirved64 Mirved64 changed the title Feat/landing third step feat/landing-third-step Aug 9, 2023
@Mirved64 Mirved64 changed the base branch from master to feat/landing-second-step August 9, 2023 06:54
…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
@Mirved64
Copy link
Contributor Author

@Nelfimov

Скоуп
Анимация параллакса в проекте

Описание
Не могу понять как сделать эффект параллакса как в макете.
Не понимаю до конца саму анимацию параллакса (как секция при прокрутке наезжает на предыдущую секцию).

Мои действия

  • Смотрел примеры - там нет этой анимации.
  • Пробовал использовать @atls-ui-proto/parallax, застопорился на настройке провайдера - не получилось разобраться с ui, забросил.
  • Решил использовать библиотеку react-scroll-parallax.
  • Подключил библиотеку, обернул App в ParallaxProvider, использовал компонент Parallax - работает.
  • Пробовал настроить анимацию, но не до конца понимаю как сделать нужную анимацию.
  • Читал документацию к библиотеке, нагуглил статью про библиотеку.
  • Не смог найти нужную анимацию (как секция при скролле наезжает на предыдущую секцию).

Вопросы
Какие параметры меняются при анимации? (translateY? тогда получается, что контент секции наезжает на предыдущую, но потом идет очень большой gap до следующей)

Какой механизм анимации?

BREAKING CHANGE: incorrect parallax props
@Nelfimov
Copy link
Contributor

Давай пока начнем с очевидных фиксов:

  1. перейди на % параметры в translateY
  2. параллакс должен распространяться на элементы внутри background
  3. параллакс как эффект по дизайну применяется только там, где есть фон - в нашем случае это синий. То есть будто белый наезжает на синий.

@Mirved64 Mirved64 requested a review from oxiqod August 12, 2023 13:56
@Mirved64 Mirved64 requested a review from oxiqod August 14, 2023 09:44
@Mirved64 Mirved64 requested a review from oxiqod August 15, 2023 08:08
@oxiqod oxiqod requested a review from Nelfimov August 15, 2023 08:40
Copy link
Contributor

@Nelfimov Nelfimov left a comment

Choose a reason for hiding this comment

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

Есть UI баги:

  1. верхний навбор ушел за вьюпорт:
Screenshot 2023-08-15 at 13 16 49
  1. Карточки не адаптивны - текст выходит за пределы:

image

  1. Список не выравнен по центру вертикали:

image

  1. Карточки, которые в мобильном виде прокручиваются свайпом, вплотную прилегают к вьюпорту слева и справа. Должны быть выровнены по верхним статичным

image

@Mirved64
Copy link
Contributor Author

@Nelfimov

Странно, у меня кроме адаптивности карточек всё хорошо. Смотрел в хроме и мозиле, точно ветка третьего степа.

  • навбар на месте
Details

image

image

  • список правильно отображается
Details

image

image

  • Карточки, которые в мобильном виде прокручиваются свайпом с зазорами по бокам
Details

Screencast.from.15.08.2023.19.59.59.webm
Screencast.from.15.08.2023.20.00.39.webm

@Mirved64 Mirved64 requested a review from Nelfimov August 15, 2023 16:19
Copy link
Contributor

@Nelfimov Nelfimov left a comment

Choose a reason for hiding this comment

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

Необходимы правки.

landing/entrypoints/renderer/package.json Show resolved Hide resolved

<Box flexDirection={['column', 'row']}>
<Column order={[3, 0]} flexBasis={[335, 1160]} flexGrow='1'>
<Card
Copy link
Contributor

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' })} />
Copy link
Contributor

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/**/**/**/**/*"]
Copy link
Contributor

Choose a reason for hiding this comment

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

А зачем такая вложенность в паттерне?

@Mirved64 Mirved64 requested a review from Nelfimov August 16, 2023 12:21
Comment on lines +16 to +18
import { CardsMaterials } from './cards'
import { CardsSwiper } from './cards'
import { CardsLearning } from './cards'
Copy link
Member

Choose a reason for hiding this comment

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

я, как и ранее - не рекомендовал бы косметические отличия выдавать за новую сущность, на 7 этапе будет бобо)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

У меня тут три разных компановки карточек.

Т.е. сущность одна - карточка, просто в одном случае показывает карточки с категорией учебный материал, в другом карточки собираются в свайпер, в третьем карточки с категорией обучение или мини-курс.

Copy link
Member

Choose a reason for hiding this comment

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

а должна быть одна карточка, которая где-либо расширяется и переиспользуется

Comment on lines +11 to +35
<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'
Copy link
Member

Choose a reason for hiding this comment

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

зачем тут везде инлайны выставленны?

Comment on lines +3 to +23
id: 0,
question: 'Подойдёт ли мне профессия?',
},
{
id: 1,
question: 'Подойдёт ли мне профессия?',
},
{
id: 2,
question: 'Подойдёт ли мне профессия?',
},
{
id: 3,
question: 'Подойдёт ли мне профессия?',
},
{
id: 4,
question: 'Подойдёт ли мне профессия?',
},
{
id: 5,
Copy link
Member

Choose a reason for hiding this comment

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

к чему копипаста? можно ведь прогнать через цикл один объект

Base automatically changed from feat/landing-second-step to master August 17, 2023 18:09
@TorinAsakura TorinAsakura merged commit b33f75f into master Aug 21, 2023
6 checks passed
@TorinAsakura TorinAsakura deleted the feat/landing-third-step branch August 21, 2023 10:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Landing - Third Step
4 participants