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.

Sign up for GitHub

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

Jump to bottom

enhance(client): MkNoteのリアクションの表示数は16に制限 #9841

Merged
merged 22 commits into from Feb 11, 2023
Merged

enhance(client): MkNoteのリアクションの表示数は16に制限 #9841

merged 22 commits into from Feb 11, 2023

Conversation

tamaina
Copy link
Member

@tamaina tamaina commented Feb 9, 2023 •
edited

Resolve #6144
Resolve #5684

What

MkNote（タイムライン上のノート表示）で、

  • リアクションの表示数は16に制限
  • あふれた場合は横に「もっと！」を表示するように

また、リアクションの表示順序はリアクション数の多い順に並べ替えるように

Why

  • リアクション数が多すぎるとまずい
  • リアクション総数を表示

@github-actions github-actions bot added the packages/frontend Client side specific issue/PR label Feb 9, 2023
@rinsuki
Copy link
Contributor

rinsuki commented Feb 9, 2023

また、リアクションの表示順序はリアクション数の多い順に並べ替えるように

これ実装ちゃんと読んでないしテストもしてないので完全に予想ですが、ストリーミングで更新されると押そうと思ったリアクションが動いたりして微妙そうじゃないですか？(一回表示したら変わらないようになってたらごめん)

@tamaina
Copy link
Member Author

tamaina commented Feb 9, 2023

これ実装ちゃんと読んでないしテストもしてないので完全に予想ですが、ストリーミングで更新されると押そうと思ったリアクションが動いたりして微妙そう

微妙かな（微妙か）

一回表示したら変わらないようになってたら

そんなことはしていない()

@tamaina
- Layout Shiftが起こらないように
f8e4d40 
- 自分のリアクションは必ずつける
@tamaina
Copy link
Member Author

tamaina commented Feb 9, 2023

Layout Shiftを起こさないようなふうにした

@tamaina
Copy link
Member Author

tamaina commented Feb 9, 2023

https://misskey.io/notes/7zzafqsm9a が

image

こうなる

@acid-chicken
Copy link
Member

(i) ボタンは現状 - があるところと置き換えていい気がする

@pantasystem pantasystem mentioned this pull request Feb 9, 2023
Closed
@acid-chicken
Copy link
Member

あと nice to have だけど、リアクションが全部表示されない今こそ総リアクション数表示していいんじゃないと思ったりしている

@tamaina
Copy link
Member Author

tamaina commented Feb 9, 2023

(i) ボタンは現状 - があるところと置き換えていい気がする

やってみた

@tamaina
Copy link
Member Author

tamaina commented Feb 9, 2023

フッターの数字って値が大きくなるとモバイルやデッキの表示が2段になってしまうので、あんまり追加したくない感じがある

@acid-chicken
Copy link
Member

フッターの数字って値が大きくなるとモバイルやデッキの表示が2段になってしまうので、あんまり追加したくない感じがある

添付画像見るとなんか現時点で既にそうなってない？

@tamaina
Copy link
Member Author

tamaina commented Feb 9, 2023

添付画像見るとなんか現時点で既にそうなってない？

それも含めてなんとかしたいのでこのPRには含めないかなという感じ

@acid-chicken
Copy link
Member

把握

@tamaina tamaina requested a review from syuilo February 9, 2023 08:40
@tamaina @acid-chicken
refactor
3bcc392 
Co-authored-by: acid-chicken <root@acid-chicken.com>
@futchitwo
Copy link
Contributor

(i) ボタンは現状 - があるところと置き換えていい気がする

  • リアクションした時としていない時で別の動作になるのは良くない
  • リアクションしていないとリアクションの一覧を見れない（ノートアクションのボタンからも飛べるが）
  • リアクション解除ボタンがなくなったのでリアクション解除方法がわかりにくくなった
    などのデメリットがあると思います

逆にiボタンがあることで、表示されてないリアクションがもっとあるということに気づけると思います

あと nice to have だけど、リアクションが全部表示されない今こそ総リアクション数表示していいんじゃないと思ったりしている

上手くいえないので書き殴り
総リアクション数が可視化されると今のMisskeyの「数字ではなくリアクションの一つ一つで承認欲求が満たされる」みたいなリアクションが持つ緩い雰囲気が少し減り、twitterのいいねのようにSNS疲れ、「いいね」依存症を誘発してしまいそう
（リノートの数も可視化されるが、これは一人で何回も行えると言った性質の違いや、そもそも拡散された回数なのでリアクションやいいねよりも承認欲求には影響しない？・基本的にリノートよりリアクション数の方が大きくなるので嫉妬の感情が沸きやすくなる）

@acid-chicken
Copy link
Member

などのデメリットがあると思います

これは私の認識が甘かったです

逆にiボタンがあることで、表示されてないリアクションがもっとあるということに気づけると思います

これが（というか (i) が）悩ましくて、リアクションの続きに (i) を使っても … を使っても詳細の意味するスコープがリアクションなのかノート自体なのか分かりにくくなってしまう問題があります

sketch note
リアクション背景の縁が重なったような感じにする
→ 双方の背景コントラスト比が低いテーマが多くてそもそも見えない
 → いっそのことテキストでもいいかもしれない
いっそのこと二段にしてしまう
→ リアクションが多くない平常時にごちゃごちゃする

@tamaina
Copy link
Member Author

tamaina commented Feb 10, 2023

詳細の意味するスコープがリアクションなのかノート自体なのか

リアクションが存在しなければiは表示されないのでそれでわかるようになっているつもり

@tamaina
Copy link
Member Author

tamaina commented Feb 10, 2023

すべてのノートに対してリアクションがついていると多少の混乱はあるかも

@syuilo
Copy link
Member

syuilo commented Feb 10, 2023

(i) はリアクションの種類が16を超えたときに末尾に表示するので良いと思った

@syuilo
Copy link
Member

syuilo commented Feb 10, 2023

あと (i) じゃなくて ... でも良さそう

@tamaina
Copy link
Member Author

tamaina commented Feb 10, 2023

 ...

ノートメニューの…と丸かぶりするのでダメかな

@tamaina
Copy link
Member Author

tamaina commented Feb 10, 2023

リアクションの種類が16を超えたとき

実装面倒かなと思ったけど普通にできるか

@tamaina
Copy link
Member Author

tamaina commented Feb 10, 2023

image

ふむ

@tamaina
Copy link
Member Author

tamaina commented Feb 10, 2023

  • opacity: .8を指定
  • MkNoteDetailedにもつけた

image

@syuilo
Copy link
Member

syuilo commented Feb 10, 2023

個人的にはちょっと分かりにくい
単に「もっと見る」という案も提出

@tamaina tamaina changed the title enhance(client): MkNoteのリアクションの表示数は16に制限・リアクションの横の？ボタンでリアクション詳細 enhance(client): MkNoteのリアクションの表示数は16に制限・リアクションの横の(i)ボタンでリアクション詳細へ Feb 10, 2023
@tamaina
Copy link
Member Author

tamaina commented Feb 10, 2023

単に「もっと見る」という案

いいと思うけどリアクション総数表示つけたらやめられなくなってきてしまった

@acid-chicken
Copy link
Member

単に「もっと見る」という案も提出

現在は自分もこれ派

@tamaina tamaina changed the title enhance(client): MkNoteのリアクションの表示数は16に制限・リアクションの横の(i)ボタンでリアクション詳細へ enhance(client): MkNoteのリアクションの表示数は16に制限 Feb 11, 2023
@tamaina
Copy link
Member Author

tamaina commented Feb 11, 2023

image

もっと！にした

@syuilo
Copy link
Member

syuilo commented Feb 11, 2023

YOSASOU

@EbiseLutica
Copy link
Member

めっちゃいい

@acid-chicken
Copy link
Member

もっと！破線にしようぜ

@tamaina
Copy link
Member Author

tamaina commented Feb 11, 2023

破線に

うるさくなるかなと思ったけどそんなことはなかったので採用

image

@syuilo syuilo merged commit 6f33be6 into misskey-dev:develop Feb 11, 2023
@syuilo
Copy link
Member

syuilo commented Feb 11, 2023

👍🏻

@tamaina tamaina mentioned this pull request Mar 1, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@acid-chicken acid-chicken acid-chicken left review comments

@syuilo syuilo Awaiting requested review from syuilo

Assignees
No one assigned
Labels
packages/frontend Client side specific issue/PR
Projects
None yet
Milestone
No milestone
6 participants
@tamaina @rinsuki @acid-chicken @futchitwo @syuilo @EbiseLutica