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

Commitせずにmergeしようとする時のalert文が読まれづらそう #308

Open
foxytanuki opened this issue May 31, 2019 · 3 comments

Comments

@foxytanuki
Copy link
Collaborator

概要

Screen Shot 2019-05-31 at 19 27 46
alertの文章は適当に読み飛ばしちゃう(と個人的には思う)

詳細

この状況下では、ユーザーの視点はCheck Diffボタンに行っているので、Alertはその近くに出すか、強引に視点移動を促す必要があると思う

以下がよさそう?
視点を極力動かさない⇢ToolTip
視点を強引に移動⇢背景ぼかしてモーダル

@foxytanuki
Copy link
Collaborator Author

現状のユーザーの行動としては
alertでる⇢とりあえず無害そうなcancel⇢なにも起きない⇢OK押すか
だと思っている
文章は読まない

@bokusunny
Copy link
Owner

@yuta-tsurusaki
ありがとう、こういう意見めっちゃ助かる!

@foxytanuki
Copy link
Collaborator Author

foxytanuki commented May 31, 2019

そもそも、commitしていない時にCheck diffは押せないようにする方が1アクション減ってやさしいかも

[alert]
Checkdiff押下 ⇢ alert ⇢ commit
[checkdiff押せない]
checkdiff押せない..?あ、commitいるのか ⇢ commit

commit周りのシステムがどうなっているかわからんからなんとも言えないが

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

No branches or pull requests

2 participants