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

通知欄のフォローボタンが消えた? #13148

Open
1 task
kakkokari-gtyih opened this issue Feb 2, 2024 · 35 comments
Open
1 task

通知欄のフォローボタンが消えた? #13148

kakkokari-gtyih opened this issue Feb 2, 2024 · 35 comments
Labels
packages/frontend Client side specific issue/PR

Comments

@kakkokari-gtyih
Copy link
Contributor

kakkokari-gtyih commented Feb 2, 2024

💡 Summary

🥰 Expected Behavior

image

ボタンが出る

🤬 Actual Behavior

image

出ない

📝 Steps to Reproduce

  1. 誰かからフォローしてもらう
  2. 通知欄を見る

💻 Frontend Environment

* Model and OS of the device(s): Windows 11 22H2
* Browser: Chrome 121.0.6167.87 (Official Build)
* Server URL: misskey.io
* Misskey: 2024.2.0-io.1 (beta.8)

🛰 Backend Environment (for server admin)

* Installation Method or Hosting Service:
* Misskey:
* Node:
* PostgreSQL:
* Redis:
* OS and Architecture:

Do you want to address this bug yourself?

  • Yes, I will patch the bug myself and send a pull request
@kakkokari-gtyih kakkokari-gtyih added the ⚠️bug? This might be a bug label Feb 2, 2024
@acid-chicken
Copy link
Member

#12926 (comment)

@syuilo
Copy link
Member

syuilo commented Feb 3, 2024

消した

@syuilo syuilo closed this as not planned Won't fix, can't repro, duplicate, stale Feb 3, 2024
@kakkokari-gtyih
Copy link
Contributor Author

なるほど
別にユーザー名にマウスオーバーしたりすればUserInfo出せるのでそこまで操作性は損なわれないわね

@tamaina tamaina reopened this Feb 3, 2024
@tamaina
Copy link
Member

tamaina commented Feb 3, 2024

ほしい

@tamaina
Copy link
Member

tamaina commented Feb 3, 2024

(フォローしているかどうかのエンドポイントを作ってほしい

@acid-chicken
Copy link
Member

acid-chicken commented Feb 3, 2024

個人的にはいらない(誤タップするのが嫌でカスタム CSS で消してた)
そもそもプロフも見ずにほいほいフォローするのはおすすめできなさそう

@kakkokari-gtyih
Copy link
Contributor Author

(フォローしているかどうかのエンドポイントを作ってほしい

そのためだけに作るのは微妙かも

@tamaina
Copy link
Member

tamaina commented Feb 3, 2024

そもそもプロフも見ずにほいほいフォローするのはおすすめできなさそう

というか、このボタンは自分でフォローしているかどうか(フォローバックの必要性があるか)を判断するために使いたいのでこのボタンを押してフォローするわけではない

そのためだけに作るのは微妙かも

getRelationをするエンドポイントはあってよくない?

@tamaina
Copy link
Member

tamaina commented Feb 3, 2024

getRelationをするエンドポイントはあってよくない?

すでにあんだわ

https://github.com/misskey-dev/misskey/blob/d8bdbd53ed9366b6578c0f2f2a44c5efa4b887a9/packages/backend/src/server/api/endpoints/users/relation.ts#L143

@acid-chicken
Copy link
Member

そもそもプロフも見ずにほいほいフォローするのはおすすめできなさそう

というか、このボタンは自分でフォローしているかどうか(フォローバックの必要性があるか)を判断するために使いたいのでこのボタンを押してフォローするわけではない

じゃあ「フォローバックされました」とかの方が良さそう

@tamaina
Copy link
Member

tamaina commented Feb 3, 2024

じゃあ「フォローバックされました」とかの方が良さそう

デカデカとフォローボタンがあった方がフォローしていないことがわかりやすい

@acid-chicken acid-chicken added packages/frontend Client side specific issue/PR and removed ⚠️bug? This might be a bug labels Feb 3, 2024
@acid-chicken
Copy link
Member

acid-chicken commented Feb 3, 2024

誤タップするのが嫌

まあそこまでこだわりはないけど

@tamaina
Copy link
Member

tamaina commented Feb 3, 2024

(設定できるようにすればいい気もするけど、そんな誤タップするんか…?

@syuilo
Copy link
Member

syuilo commented Feb 3, 2024

割と誤タップするから無くなって良かったみたいな声は聞くわね

@acid-chicken
Copy link
Member

スクロール要素の中にワンタップでオペレーションが発効するボタンを置くのはあまりいい UI ではなさそう(なので Twitter のいいねボタンとかも好きじゃなかった)

@tamaina
Copy link
Member

tamaina commented Feb 3, 2024

ちょっと賛同できかねるなぁ、ハイパーリンクまで全部確認UIが必要ということになるのは単に面倒

@acid-chicken
Copy link
Member

ハイパーリンクはオペレーションではないので

@tamaina
Copy link
Member

tamaina commented Feb 3, 2024

意図しないページの移動はオペレーションより苦痛なので

スマホで誤タップで困ったことないと言おうとしたがYouTubeのプレイリスト追加はなぜか誤タップが多いわね

@acid-chicken
Copy link
Member

意図しないページ遷移はまあフラストレーションにはなれど恐怖にはつながらないので別の課題かも
誤操作により取り消せない行為を行う可能性があるという潜在的恐怖があると個人的にあまり触りたくなくなる

@tamaina
Copy link
Member

tamaina commented Feb 3, 2024

(スクロールに一発操作があって欲しくないならフォローボタン自体に確認ダイアログをつけるとかの方がいい気がした)

@tamaina
Copy link
Member

tamaina commented Feb 3, 2024

別にフォローは取り消せるのであんまり恐怖はないわね

@acid-chicken
Copy link
Member

(スクロールに一発操作があって欲しくないならフォローボタン自体に確認ダイアログをつけるとかの方がいい気がした)

それはそうかも

@tamaina
Copy link
Member

tamaina commented Feb 6, 2024

やるか…

@tamaina
Copy link
Member

tamaina commented Feb 6, 2024

フォローボタンの確認ダイアログはつけるとして、オプションで確認ダイアログを切れるようにするかどうか悩みどころ

@syuilo
Copy link
Member

syuilo commented Feb 6, 2024

実装する場合は通知内で表示される場合は有り、それ以外の場所では無しで良さそう
ただそもそも通知内のフォローボタンの必要性には疑問がある

@syuilo
Copy link
Member

syuilo commented Feb 6, 2024

フォローボタンというよりフォローバックボタンならあっても良さそうだけど

@anatawa12
Copy link
Member

フォローしてるつもりだったけどフォローしてなかったというときにフォローされたタイミングで気づいてフォローバックしやすいと嬉しい

@tamaina
Copy link
Member

tamaina commented Feb 6, 2024

フォローボタンというよりフォローバックボタンならあっても良さそうだけど

フォローしていないアカウントからフォローされた場合だけ白いフォローボタンを表示する機能だったんだけど、わざわざ「フォローバック」に文言を変える必要ある?

@tamaina
Copy link
Member

tamaina commented Feb 6, 2024

(ここの議論、積極的にフォローバックする人とそうではない人で意見が真っ二つに分かれている感じがある

@tamaina
Copy link
Member

tamaina commented Feb 6, 2024

ただそもそも通知内のフォローボタンの必要性には疑問がある

通知内にフォローボタンを表示するかどうかもオプションにするか

@tamaina tamaina assigned tai-cha and unassigned tai-cha Feb 8, 2024
@kabo2468
Copy link
Contributor

kabo2468 commented Feb 8, 2024

related to #6020

@acid-chicken
Copy link
Member

実装する場合は通知内で表示される場合は有り、それ以外の場所では無しで良さそう

全部ついてて欲しい(プロフィール画面のフォローボタンもメニュー開く際にタップミスしそうで怖いため)

@anatawa12
Copy link
Member

結構前の発言になるけど

別にフォローは取り消せるのであんまり恐怖はないわね

に関しては通知という形で残るので少し恐怖感あります

@tamaina
Copy link
Member

tamaina commented Feb 18, 2024

アンケートを取ったところフォローバックボタンが必要という意見が結構多い

https://p1.a9z.dev/notes/9pv87ghf9q

@kakkokari-gtyih
Copy link
Contributor Author

とりあえず確認ダイアログを表示するようにするか

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages/frontend Client side specific issue/PR
Projects
None yet
Development

No branches or pull requests

7 participants