-
Notifications
You must be signed in to change notification settings - Fork 12
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(spindle-ui): 通知系コンポーネントの使い分けについて詳細追加 #926
base: main
Are you sure you want to change the base?
Conversation
reg-suit detected visual differences. Check this report, and review them.
How can I change the check status?If reviewers approve this PR, the reg context status will be green automatically. |
Visit the preview URL for this PR (updated for commit 8409838): https://ameba-spindle--pr926-feature-update-notif-o20dkq8j.web.app (expires Sat, 26 Oct 2024 06:58:49 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: e7521619a2dd5c653490c8246e81ec2a5c8f1435 |
### 通信を伴うデータ更新のフィードバック | ||
|
||
- 成功:[Toast - Confirmation](/docs/toast--confirmation)で、成功した事実のみを伝える | ||
- 失敗:[InlineNotification - Error Emphasis](/docs/inlinenotification--error-emphasis)で、失敗した理由とユーザーがすべきアクションを伝える | ||
- 一連の動作のゴールがある場合:完了画面に遷移させる。例: | ||
- 登録完了時 | ||
- 「元の画面に戻る」動作をユーザーにさせたい時 |
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.
Snackbarはデータ更新のフィードバックには使わないので、使ってはいけない、のように書く方が良いでしょうか 💭
This comment was marked as resolved.
This comment was marked as resolved.
eb69b6a
to
7c4e9e4
Compare
This comment was marked as resolved.
This comment was marked as resolved.
7c4e9e4
to
80b5100
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.
一旦、大枠だけコメントさせていただきました! 🙏🏻 (細かい部分は修正次第で変わると思うので一旦スルーしています)
packages/spindle-ui/src/InlineNotification/InlineNotification.stories.mdx
Outdated
Show resolved
Hide resolved
packages/spindle-ui/src/InlineNotification/InlineNotification.stories.mdx
Outdated
Show resolved
Hide resolved
packages/spindle-ui/src/InlineNotification/InlineNotification.stories.mdx
Outdated
Show resolved
Hide resolved
packages/spindle-ui/src/InlineNotification/InlineNotification.stories.mdx
Outdated
Show resolved
Hide resolved
#926 (comment) と #926 (comment) について、たしかになので @MasatoHonda とSlackで詰めます:pray:(内部資料参照したいのでslackにて・・) |
d135006
to
4fd3428
Compare
WAIのToast/SnackBarへの公式見解が出されたことを受けて内容を大幅に見直しました。まずはほんでぃと来週以降で相談しつつ修正を加えていく予定です。その後再レビュー依頼します!(内容にご意見いただくのは今でも全然!) |
4fd3428
to
8409838
Compare
@herablog @itsminadesu @yasuda-shin デザイナーチェックokだったので、再レビューお願いします〜〜!(PRガッツリ描き直しています 🙇 ) |
2.2.1 コンテンツに制限時間を設けない の観点から、長い間アクセシビリティ上の懸念が払拭できなかったSnackBar, Toastについて、WAIから公式見解が示されたため、AmebaとしてもSnackBarとToastの 適切な 利用法について言及したいと思います。
詳細
主にモバイルアプリケーションで一般的になったSnackBarやToastコンポーネントは、制限時間が来ると自動で消えてしまうので、2.2.1の達成基準で問題になるのでは?と考えられていました。しかし、時間制限のない代替手段がある場合は、必ずしも時間調整できる必要はないとのことです。
という段落が2024/6に追加されていました。
翻訳アプリで直訳:
無条件でokというわけではなく、条件付きなことに注意が必要なため、Spindle UIとしても用途について言及する必要があります。
ユーザーがToastを見逃しても支障がなく(=代替手段があり)、見逃さなかった人にとっては使い勝手が向上するもの、です。
一方で、画面拡大時など、画面外に表示されてしまうことで通知を見逃してしまうというアクセシビリティの懸念はあるため、SnackBarとToastを使う際には下記2つの基準をクリアすることを明記します。
上記を踏まえて、Toast, SnackBar, InlineNotification, Dialogの使い分けを明確にするためにコンポーネントの説明部分と使い分けについて記載しました。
気になりポイント
スケジュール
急ぎではないです 🙏