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.

Sign up for GitHub

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

feat!: ErrorScreen の footer props を omit する #4714

Merged
merged 9 commits into from
Jun 14, 2024
Merged

feat!: ErrorScreen の footer props を omit する #4714

Show file tree
Hide file tree
Changes from 5 commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
7912177
feat: ErrorScreen の `footer` props を deprecated にした
oti Jun 11, 2024
959b6d9
feat: ErrorScreen の Storybook の `full` を `all` にして内容を調整した
oti Jun 11, 2024
34dcd80
feat: `footer` props利用の例をトルツメした
oti Jun 11, 2024
4579650
feat: `without children` の例をトルツメした
oti Jun 11, 2024
132ecc9
feat: `all` の文言を無味無臭にした
oti Jun 11, 2024
3cb4633
feat: ErrorScreen から footer props を omit した
oti Jun 13, 2024
aef9614
Merge branch 'master' into SHRUI-987
oti Jun 13, 2024
9014b5e
feat: 不要になった内部コンポーネントを削除した
oti Jun 13, 2024
093d132
Merge branch 'master' into SHRUI-987
oti Jun 14, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
49 changes: 18 additions & 31 deletions packages/smarthr-ui/src/components/ErrorScreen/ErrorScreen.stories.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import { StoryFn } from '@storybook/react'
import React from 'react'

import { Footer } from './Footer'
import { Text } from '../Text'

import { ErrorScreen } from '.'

Expand All @@ -13,42 +13,29 @@ export default {
},
}

export const Full: StoryFn = () => (
export const All: StoryFn = () => (
<ErrorScreen
title="SmartHR は現在メンテナンス中です"
className="shr-w-full"
title="タイトル"
links={[
{
label: 'SmartHR お知らせ',
url: 'http://example.com',
label: 'リンク1',
url: '/',
},
{
label: 'リンク2',
url: '/',
},
{
label: 'リンク3',
url: '/',
target: '_blank',
},
]}
>
<p>
いつも SmartHR をご利用いただきありがとうございます。
<br />
ただいまメンテナンスのため、一時サービスを停止しております。
<br />
ご迷惑をおかけいたしますが、ご理解のほどよろしくお願いいたします。
</p>
<Text as="p" className="shr-text-center">
メッセージ
</Text>
</ErrorScreen>
)
Full.storyName = 'full'

export const WithoutChildren: StoryFn = () => (
<ErrorScreen
title="サンプルタイトル"
links={[
{
label: 'ホームへ',
url: 'http://example.com',
},
]}
/>
)
WithoutChildren.storyName = 'without children'

export const WithFooter: StoryFn = () => (
<ErrorScreen links={[{ label: 'ホームへ', url: 'http://example.com' }]} footer={<Footer />} />
)
WithFooter.storyName = 'with footer'
All.storyName = 'all'
2 changes: 1 addition & 1 deletion packages/smarthr-ui/src/components/ErrorScreen/ErrorScreen.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ type Props = {
}>
/** 表示するコンテンツ */
children?: ReactNode
/** フッター */
/** @deprecated フッター指定は非推奨です。リンクの表示が必要な場合は links を使用してください。 */
Copy link
Contributor

Choose a reason for hiding this comment

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

使われていないのなら非推奨化じゃなくて即削除しちゃっても良いかなと思いましたがどうでしょう？
逆に、残すのであれば非推奨であろうと Storybook で確認できてほしい気がしました。

footer?: ReactNode
/** コンポーネントに適用するクラス名 */
className?: string
Expand Down