Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: RadioButton の Story を見直し #5063

Merged
merged 4 commits into from
Oct 31, 2024
Merged

Conversation

uknmr
Copy link
Collaborator

@uknmr uknmr commented Oct 28, 2024

関連URL

https://smarthr.atlassian.net/browse/SHRUI-1103

概要

#4949 の方針に沿って RadioButton と RadioButtonPanel の Story を見直しました。

変更内容

確認方法

Storybook や Chromatic で確認してください。

@uknmr uknmr requested a review from a team as a code owner October 28, 2024 00:33
@uknmr uknmr changed the title Improve radio button story docs: RadioButton の Story を見直し Oct 28, 2024
Copy link

pkg-pr-new bot commented Oct 28, 2024

Open in Stackblitz

pnpm add https://pkg.pr.new/kufu/smarthr-ui@5063

commit: 2ca82a5

Copy link
Contributor

@s-sasaki-0529 s-sasaki-0529 left a comment

Choose a reason for hiding this comment

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

一点だけコメントしました!あとは良さそうです!

render: (args) => <RadioButtonPanel {...args} />,
args: {
children: 'ラジオボタンパネル',
},
Copy link
Contributor

Choose a reason for hiding this comment

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

ここ、as の定義がないと Playground とか Controls 全般でクラッシュしちゃいます ><

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

修正しました! 8e3af66

@uknmr uknmr requested a review from s-sasaki-0529 October 31, 2024 02:47
@uknmr uknmr merged commit f064f85 into master Oct 31, 2024
14 checks passed
@uknmr uknmr deleted the improve-RadioButton-story branch October 31, 2024 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.

2 participants