Skip to content

Conversation

@lentsd
Copy link
Contributor

@lentsd lentsd commented Nov 10, 2025

Updated Share block story for better storybook ux

@gravity-ui-bot
Copy link
Contributor

Preview is ready.

@gravity-ui-bot
Copy link
Contributor

Playwright Test Component is ready.

title: 'Blocks/Share',
argTypes: {
title: {
control: {type: 'text'},
Copy link
Collaborator

Choose a reason for hiding this comment

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

Let's make sure that these controls are pulled up automatically. If they did not appear automatically, it is possible that this is due to Pick<>, different names of the file and the component in the file.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Deleted title from argTypes, but left for items because it looks better with control type: 'check'


const DefaultTemplate: StoryFn<ShareBLockModel> = (args) => (
<PageConstructor content={{blocks: [args]}} />
<PageConstructor content={{blocks: [blockTransform(args)]}} />
Copy link
Collaborator

Choose a reason for hiding this comment

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

Let's remove the PageConstructor here and just make a Share component. Most likely, this will fix the problem with the controls.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed PageConstructor

@lentsd lentsd force-pushed the chore/update-share-storybook branch from c092e24 to e1965bc Compare November 11, 2025 11:11
@lentsd lentsd requested a review from aeksandla November 11, 2025 13:04
@lentsd lentsd merged commit 203ca7b into main Nov 12, 2025
4 checks passed
@lentsd lentsd deleted the chore/update-share-storybook branch November 12, 2025 08:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants