-
Notifications
You must be signed in to change notification settings - Fork 141
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
feat: AppHeader コンポーネントを追加 #5203
Conversation
commit: |
8d7b290
to
c15e29d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
一件コメントしました〜!
プロダクトに入れて動かして見ましたが、コメントしたところ以外は良さそうでした!ありがとうございます! 🙏
|
||
import { AppLauncher } from './AppLauncher' | ||
import { Navigation } from './Navigation' | ||
import { UserInfo } from './UserInfo.tsx' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ここの .tsx
を消す必要がありそうです! 🙏
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ほんまですね…!修正しました! 920cb76
ad7d867
to
0edb432
Compare
0edb432
to
920cb76
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
対応あざます〜 🙏
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
gogo~
関連URL
🍐
概要
AppHeader コンポーネントを追加したい。
AppHeader コンポーネントは、Header と AppNavi を結合したコンポーネントです。各プロダクトのヘッダーの表示を共通化させるために存在します。
詳しくは コンポーネントの README を参照してください。
変更内容
AppHeader の追加
確認方法
Storybook で確認してください。
https://63d0ccabb5d2dd29825524ab-vwkuynyozq.chromatic.com/?path=/story/navigation%EF%BC%88%E3%83%8A%E3%83%93%E3%82%B2%E3%83%BC%E3%82%B7%E3%83%A7%E3%83%B3%EF%BC%89-appheader--playground