Skip to content

feat(plasma-temple): Custom title component in GalleryPage #1119

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

VladislavPetyukevich
Copy link
Contributor

@VladislavPetyukevich VladislavPetyukevich commented Feb 22, 2022

Возможность передавать свой компонент для заголовков в GalleryPage. Необходимо для возможности их кастомизации

📦 Published PR as canary version: Canary Versions

✨ Test out this PR locally via:

npm install @sberdevices/demo-canvas-app@0.60.0-canary.1119.32daae97ffb79d411d316ff19a796065efd2ac39.0
npm install @sberdevices/plasma-b2c@1.42.0-canary.1119.32daae97ffb79d411d316ff19a796065efd2ac39.0
npm install @sberdevices/plasma-core@1.50.0-canary.1119.32daae97ffb79d411d316ff19a796065efd2ac39.0
npm install @sberdevices/plasma-icons@1.68.0-canary.1119.32daae97ffb79d411d316ff19a796065efd2ac39.0
npm install @sberdevices/plasma-temple@1.32.0-canary.1119.32daae97ffb79d411d316ff19a796065efd2ac39.0
npm install @sberdevices/plasma-ui@1.83.0-canary.1119.32daae97ffb79d411d316ff19a796065efd2ac39.0
npm install @sberdevices/plasma-web@1.77.0-canary.1119.32daae97ffb79d411d316ff19a796065efd2ac39.0
npm install @sberdevices/plasma-sb-utils@0.49.0-canary.1119.32daae97ffb79d411d316ff19a796065efd2ac39.0
npm install @sberdevices/showcase@0.98.0-canary.1119.32daae97ffb79d411d316ff19a796065efd2ac39.0
npm install @sberdevices/plasma-ui-docs@0.44.0-canary.1119.32daae97ffb79d411d316ff19a796065efd2ac39.0
npm install @sberdevices/plasma-web-docs@0.34.0-canary.1119.32daae97ffb79d411d316ff19a796065efd2ac39.0
npm install @sberdevices/plasma-website@0.32.0-canary.1119.32daae97ffb79d411d316ff19a796065efd2ac39.0
# or 
yarn add @sberdevices/demo-canvas-app@0.60.0-canary.1119.32daae97ffb79d411d316ff19a796065efd2ac39.0
yarn add @sberdevices/plasma-b2c@1.42.0-canary.1119.32daae97ffb79d411d316ff19a796065efd2ac39.0
yarn add @sberdevices/plasma-core@1.50.0-canary.1119.32daae97ffb79d411d316ff19a796065efd2ac39.0
yarn add @sberdevices/plasma-icons@1.68.0-canary.1119.32daae97ffb79d411d316ff19a796065efd2ac39.0
yarn add @sberdevices/plasma-temple@1.32.0-canary.1119.32daae97ffb79d411d316ff19a796065efd2ac39.0
yarn add @sberdevices/plasma-ui@1.83.0-canary.1119.32daae97ffb79d411d316ff19a796065efd2ac39.0
yarn add @sberdevices/plasma-web@1.77.0-canary.1119.32daae97ffb79d411d316ff19a796065efd2ac39.0
yarn add @sberdevices/plasma-sb-utils@0.49.0-canary.1119.32daae97ffb79d411d316ff19a796065efd2ac39.0
yarn add @sberdevices/showcase@0.98.0-canary.1119.32daae97ffb79d411d316ff19a796065efd2ac39.0
yarn add @sberdevices/plasma-ui-docs@0.44.0-canary.1119.32daae97ffb79d411d316ff19a796065efd2ac39.0
yarn add @sberdevices/plasma-web-docs@0.34.0-canary.1119.32daae97ffb79d411d316ff19a796065efd2ac39.0
yarn add @sberdevices/plasma-website@0.32.0-canary.1119.32daae97ffb79d411d316ff19a796065efd2ac39.0

@SberDevices-Eva
Copy link
Collaborator

Demo canvas app deployed!

http://plasma.sberdevices.ru/demo-canvas-app-pr-1119/

@codecov
Copy link

codecov bot commented Feb 22, 2022

Codecov Report

Merging #1119 (1548998) into master (c2b3111) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1119   +/-   ##
=======================================
  Coverage   87.97%   87.97%           
=======================================
  Files         346      346           
  Lines        4558     4558           
  Branches     1346     1346           
=======================================
  Hits         4010     4010           
  Misses        548      548           
Flag Coverage Δ
plasma-b2c 96.02% <ø> (ø)
plasma-core 94.90% <ø> (ø)
plasma-temple 73.81% <ø> (ø)
plasma-ui 94.47% <ø> (ø)
plasma-web 95.41% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
packages/plasma-core/src/mixins/addFocus.ts 90.90% <ø> (ø)
...lasma-temple/src/pages/GalleryPage/GalleryPage.tsx 91.80% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 218f847...1548998. Read the comment docs.

@SberDevices-Eva
Copy link
Collaborator

@@ -54,13 +55,15 @@ const StyledSectionTitle = styled(Headline3)<{ active: boolean }>`
interface FocusableGalleryProps {
index: number;
title?: string;
TitleComponent?: React.ComponentType<{ title: string }>;
Copy link
Contributor

Choose a reason for hiding this comment

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

Если сделать title?: React.ReactNode закроет кейс?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Да, можно рендерить компоненты в поле title внутри state. Спасибо, сразу не подумал

@SberDevices-Eva
Copy link
Collaborator

Demo canvas app deployed!

http://plasma.sberdevices.ru/demo-canvas-app-pr-1119/

@SberDevices-Eva
Copy link
Collaborator

@Yeti-or
Copy link
Contributor

Yeti-or commented Feb 24, 2022

@VladislavPetyukevich а что именно хочется кастомизировать в этих заголовках?
потому что последний раз с @gbushnev договаривались что заголовки дожны быть
SberBox – Headline3
Portal – Body3
Mobile – Body3

и никакой кастомизации больше не предполагалось

сс @LamaEats

@LamaEats
Copy link
Contributor

@VladislavPetyukevich а что именно хочется кастомизировать в этих заголовках? потому что последний раз с @gbushnev договаривались что заголовки дожны быть SberBox – Headline3 Portal – Body3 Mobile – Body3

и никакой кастомизации больше не предполагалось

сс @LamaEats

да, такой разговор был, что из вне компонент тайтла не кастомизируемый, а должен быть строго определенной типогрфике, которую указал @Yeti-or

@VladislavPetyukevich
Copy link
Contributor Author

@VladislavPetyukevich а что именно хочется кастомизировать в этих заголовках? потому что последний раз с @gbushnev договаривались что заголовки дожны быть SberBox – Headline3 Portal – Body3 Mobile – Body3
и никакой кастомизации больше не предполагалось
сс @LamaEats

да, такой разговор был, что из вне компонент тайтла не кастомизируемый, а должен быть строго определенной типогрфике, которую указал @Yeti-or

Хорошо, значит надо устранить проблему с тем, что для портала не ставится заголовок Body3. Но у заголовка нет отступа снизу. Это так должно быть? На скрине отступ между заголовком и карточкой галереи
Снимок экрана 2022-02-24 в 14 52 54
Мне нужно добавить отступ снизу у заголовка. Будет ли хорошим решением добавить отступ сверху у всех карточек?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants