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

デザインパターン「視覚的グルーピング」を追加 #1355

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

tosiiu
Copy link
Contributor

@tosiiu tosiiu commented Oct 30, 2024

課題・背景

使用性チェックリスト10番の基準充足

やったこと

  • デザインパターン「視覚的グルーピング」の追加
    • PageHeadingがXXLな世界線でキャプチャを貼っていますが、これは決定事項では無いので議論次第で差し替えます(XXL化の議論はSlackで別途行います)

例として挙げている画像の妥当性がやや気になっているのでよりよい例があれば提案ほしいです

やらなかったこと

  • 複数のセクションをTabBarによって切り替える場合の複数のセクションのグルーピング手法
    • TabBar分科会の持ち物であるためWIPとしています。決定次第、記載します。

動作確認

https://deploy-preview-1355--smarthr-design-system.netlify.app/products/design-patterns/visual-grouping/

キャプチャ

Before After

Copy link

netlify bot commented Oct 30, 2024

Deploy Preview for smarthr-design-system ready!

Name Link
🔨 Latest commit 8f28c39
🔍 Latest deploy log https://app.netlify.com/sites/smarthr-design-system/deploys/672c2009f8646b00089e6709
😎 Deploy Preview https://deploy-preview-1355--smarthr-design-system.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@tosiiu tosiiu marked this pull request as ready for review October 30, 2024 05:57
@tosiiu tosiiu requested review from versionfive and a team as code owners October 30, 2024 05:57
Copy link
Contributor

@oremega oremega left a comment

Choose a reason for hiding this comment

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

サンプル画像が気になったのでコメントしました。

Copy link
Contributor

Choose a reason for hiding this comment

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

より実践で役に立つようにとレイアウトを複雑にしていると思いますが、余白以外にも矩形(Base)も追加されているので、余白が生かされていることがわかりにくくなっていると思いました。
もっとシンプルにしても余白だけでコントロールしたほうが伝わるかもと思いました。

image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

多重構造の表現に余白は向いていないよ!という文脈で例を出していたので矩形でのレイアウト例を載せていました!
ただ多重構造のDo/Don'tであることが結構分かりづらいですかね〜〜〜〜〜〜〜〜〜〜〜〜〜〜〜〜〜

Copy link
Contributor

Choose a reason for hiding this comment

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

あー、そうか、余白、矩形、罫線のだめな使い方に注意してなのですね。
Do/Don'tを使っているからDoを見たときに推奨した使い方というバイアスがあったので、より大事にしているのはだめな使い方の方っていう構成なんですかね。

Copy link
Contributor

Choose a reason for hiding this comment

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

矩形の表現を COLUMの色で囲っている例を出しているので、すべて取り払われているのはいわかんありました。ある程度はほしいなということで、イメージ貼っつけておきます。
image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

矩形の過剰な使用のDo/Don'tなので確かにもうちょっと現実的なラインに寄せても良さそうですね
変更してみます

Copy link
Contributor

@versionfive versionfive left a comment

Choose a reason for hiding this comment

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

ざっとコメントしました。
添付の画像についてSlackにてコメントしていますmm

SmartHRプロダクトの基本的な視覚的グルーピングのパターンをまとめています。

## 基本的な考え方
余白・矩形・罫線・影による視覚的グルーピングは、情報の支配関係や意味的なまとまりといった要素同士の関係性を視覚的に表現するために使います。
Copy link
Contributor

Choose a reason for hiding this comment

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

支配関係

言いたいことはわかるけど、この表現でいいのかちょっとわからない…
従属関係?主従関係?

後の文章に「影響範囲」「階層」「階層構造・内包関係」という言葉もでてくる

また、同じ階層・同じ粒度のグループが複数ある場合は、特別な理由がない限り同じ表現でグルーピングしてください。むやみに異なる表現を利用することは、ユーザーが情報を理解する障壁となります。

### 階層を深くしない
視覚的なグループの内部でさらに要素をグルーピングすることで、情報の階層構造・内包関係を表現できます。ただし、過剰に深い階層はユーザーの素早く正確な情報の理解を妨げます。
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
視覚的なグループの内部でさらに要素をグルーピングすることで、情報の階層構造・内包関係を表現できます。ただし、過剰に深い階層はユーザーの素早く正確な情報の理解を妨げます
視覚的なグループの内部でさらに要素をグルーピングすることで、情報の階層構造・内包関係を表現できます。ただし、階層が深くなるほど情報の関係性の把握が難しくなることに注意してください

「情報の理解」よりも今回は「情報の関係性の理解」かなと思って直してみたのですが、、いまいちかも

Comment on lines +35 to +37
また、情報が関連しているからといって必ずしも視覚的にグルーピングしなければならないわけではありません。

ユーザーのユースケースに基づいて情報設計やページ構成を見直し、階層が深くなることを避け、素早く理解できるレイアウトを提供してください。
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
また、情報が関連しているからといって必ずしも視覚的にグルーピングしなければならないわけではありません。
ユーザーのユースケースに基づいて情報設計やページ構成を見直し、階層が深くなることを避け、素早く理解できるレイアウトを提供してください。
また、情報が関連しているからといって必ずしも視覚的にグルーピングしなければならないわけではありません。
ユーザーのユースケースに基づいて情報設計やページ構成を見直し、階層が深くなることを避け、素早く理解できるレイアウトを提供してください。

この2段落は一連の話だと理解したので同じ段落にしました。

この1文目の意図の確認ですが、
そもそも視覚的にグルーピングしなくても、関連した情報自体には暗黙的に関連があることをユーザーは見いだせるって話ですかね?「視覚情報に頼ったグルーピングをしなくてもわかるよ」みたいなニュアンスがつたえたい?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

関連性がある情報すべてを視覚的にグルーピングする必要はなく、ユーザーにとってその関連性を理解することが有益な場合にグルーピングすべき、ということを伝えたかったです!
ただ、実際そんな例があるか改めて考えると思い当たらないのでこの一文取っちゃっても良いかなぁという気もしています

<DoAndDont
type="dont"
img={<StaticImage src={imageUrlBorderDont} alt="Dont" />}
label="どこまでの要素をどの見出しが支配するのか理解が難しい。smarthr-uiではInputやButtonなど操作可能な要素に対して罫線をつける意匠を使っているため、それと競合し操作可能な領域がわかりづらい。"
Copy link
Contributor

Choose a reason for hiding this comment

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

smarthr-uiではInputやButtonなど操作可能な要素に対して罫線をつける意匠を使っているため、それと競合し操作可能な領域がわかりづらい。

これ罫線をつかわない肝だとおもってまして、Do/Don'tの外で書いたほうがよさそうだと思いました。


セクションの内容を切り替えるコントロールを提供する場合には、[TabBar](/products/components/tab-bar/)または[SideNav](/products/components/side-nav/)を利用し以下のようにレイアウトします。

TabBarを使う場合、見出し領域にTabBarを配置し、Baseを切り替えます。見出しは省略できます。操作の対象がタブごとに異なる場合は[DropdownMenuButton](/products/components/dropdown/dropdown-menu-button/)で操作まとめます。
Copy link
Contributor

Choose a reason for hiding this comment

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

視覚上は見出しは省略するんですが、HTML的に見出し要素は必要なので、実際は<Heading visuallyHidden>の配置が必要かなと思います。

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.

3 participants