Headerが何のためのコンポーネントであるかを実態に沿ったものへ変更 #1477
Conversation
✅ Deploy Preview for smarthr-design-system ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
| --- | ||
| import ComponentPropsTable from '@/components/article/ComponentPropsTable.astro' | ||
| import ComponentStory from '@/components/article/ComponentStory.astro' | ||
|
|
||
| プロダクトをまたいでSmartHRをブラウジングしていることをユーザーが認知するためのコンポーネントです。企業アカウントの切り替えやグローバルな機能の切り替えなどを埋め込めます。 | ||
| プロダクトのアイデンディディを示し、横断的な機能の提供やプロダクトの包括的なナビゲーションの役割をしているコンポーネントです。企業アカウントの切り替えやグローバルな機能の切り替えなどを埋め込めます。 |
There was a problem hiding this comment.
プロダクトのアイデンティティが何のことか分かりづらいかも...と思いました
デザインパターンの方のヘッダーでは「アカウントやシステムの設定、およびSmartHR内の他のアプリケーションへの横断的なアクセスを提供します。」と書いているのでそれに合わせてもいいかもですかね〜。プロダクトのアイデンティティのニュアンスが失われちゃう気はしますが...
There was a problem hiding this comment.
プロダクトのアイデンティティはSmartHRロゴが担っているところなので、残すならロゴが担っていることがわかるようにしたほうよさそうです。
わたしもプロダクトを横断して共通の機能を提供するものだと思うし、ページの最上部に配置されるバーですとかもあってもいいのかな。被りはしますが、AppHeaderもそんな感じで書かれていたので。
There was a problem hiding this comment.
確認ありがとうございます
たしかにブランドアイデンディディはわかりずらいのでデザインパターンの表記に合わせようと思います！
| --- | ||
| import ComponentPropsTable from '@/components/article/ComponentPropsTable.astro' | ||
| import ComponentStory from '@/components/article/ComponentStory.astro' | ||
|
|
||
| プロダクトをまたいでSmartHRをブラウジングしていることをユーザーが認知するためのコンポーネントです。企業アカウントの切り替えやグローバルな機能の切り替えなどを埋め込めます。 | ||
| ページの最上部に配置されるバーです。アカウントやシステムの設定、およびSmartHR内の他のアプリケーションへの横断的なアクセスを提供します。企業アカウントの切り替えやグローバルな機能の切り替えなどを埋め込めます。 |
There was a problem hiding this comment.
バーという表記正しいとは思うのですがイメージが付かないなぁと感じました。「ナビゲーションのためのバー」など説明を加えるか、以下のようにバーと明記せずに説明できると良さそうです。
アカウントやシステムの設定、およびSmartHR内の他のアプリケーションへの横断的なアクセスを提供するためのコンポーネントです。ページの最上部に配置します。
| --- | ||
| import ComponentPropsTable from '@/components/article/ComponentPropsTable.astro' | ||
| import ComponentStory from '@/components/article/ComponentStory.astro' | ||
|
|
||
| プロダクトをまたいでSmartHRをブラウジングしていることをユーザーが認知するためのコンポーネントです。企業アカウントの切り替えやグローバルな機能の切り替えなどを埋め込めます。 | ||
| ページの最上部に配置されるバーです。アカウントやシステムの設定、およびSmartHR内の他のアプリケーションへの横断的なアクセスを提供します。企業アカウントの切り替えやグローバルな機能の切り替えなどを埋め込めます。 |
There was a problem hiding this comment.
企業アカウントの切り替えやグローバルな機能の切り替えなどを埋め込めます。
「アカウントやシステムの設定、およびSmartHR内の他のアプリケーションへの横断的なアクセス」と意味が重複するので消しちゃっても良いかも？
| --- | ||
| import ComponentPropsTable from '@/components/article/ComponentPropsTable.astro' | ||
| import ComponentStory from '@/components/article/ComponentStory.astro' | ||
|
|
||
| プロダクトをまたいでSmartHRをブラウジングしていることをユーザーが認知するためのコンポーネントです。企業アカウントの切り替えやグローバルな機能の切り替えなどを埋め込めます。 | ||
| ページの最上部に配置されるバーです。アカウントやシステムの設定、およびSmartHR内の他のアプリケーションへの横断的なアクセスを提供します。企業アカウントの切り替えやグローバルな機能の切り替えなどを埋め込めます。 |
There was a problem hiding this comment.
読み合わせ会のFigJamを見てきたのですがやはり、プロダクトのアイデンティティを示す側面をなんとか分かりやすく書きたいなと思いました。
難しいですが以下のような文を概要文の末尾に書くとかですかね....（本文に書かれていればdescriptionには要らないかも）
「複数のSmartHRのプロダクトを使っていたとしても同じSmartHRブランドの製品であることをユーザーが認知できるよう、共通のHeaderを提供します。」
|
@tosiiu |
|
@tosiiu |
課題・背景
https://smarthr.atlassian.net/browse/SD-900
やったこと
やらなかったこと
動作確認
キャプチャ