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

enhance(frontend): リアクションの総数を表示するか設定で選べるように #13539

Merged
merged 10 commits into from
Mar 8, 2024

Conversation

kakkokari-gtyih
Copy link
Contributor

@kakkokari-gtyih kakkokari-gtyih commented Mar 7, 2024

What

書いてあるとおり
Cherry-picked from MisskeyIO#512

Why

非表示にする設定を入れていなかったので

Additional info (optional)

ioからの変更点

  • Fix: reactionAcceptanceが「いいねのみ」の場合は強制的にカウントを表示するように
    (カウントを知る術がなくなるので)
  • リアクション総数はデフォルトOFF
    (使った感じから切っといたほうが良いと判断)

Checklist

  • Read the contribution guide
  • Test working in a local environment
  • (If needed) Add story of storybook
  • (If needed) Update CHANGELOG.md
  • (If possible) Add tests

@github-actions github-actions bot added the packages/frontend Client side specific issue/PR label Mar 7, 2024
Copy link

codecov bot commented Mar 7, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 77.51%. Comparing base (f4a5740) to head (3221970).

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #13539      +/-   ##
===========================================
- Coverage    77.68%   77.51%   -0.18%     
===========================================
  Files          185      183       -2     
  Lines        25049    24859     -190     
  Branches       484      484              
===========================================
- Hits         19460    19270     -190     
  Misses        5582     5582              
  Partials         7        7              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@kakkokari-gtyih
Copy link
Contributor Author

kakkokari-gtyih commented Mar 7, 2024

リアクション総数、デフォルト非表示でも良いかもしれない
リアクション自体に目が行かず、総数の方に目がいってしまうのでリアクションの良さが潰れる気がした

@syuilo
Copy link
Member

syuilo commented Mar 7, 2024

返信数とリノート数は隠せる必要あるかしら

@kakkokari-gtyih
Copy link
Contributor Author

kakkokari-gtyih commented Mar 7, 2024

ioから引っ張ってきたので詳細はわからない(「なんでリアクションは隠せるのに…?」となる可能性を見越した?)

@syuilo
Copy link
Member

syuilo commented Mar 7, 2024

とりあえずリアクションだけで良さそう

@kakkokari-gtyih
Copy link
Contributor Author

リアクションだけになったはず

@kakkokari-gtyih
Copy link
Contributor Author

まだだった

@kakkokari-gtyih
Copy link
Contributor Author

いけたかも

@kakkokari-gtyih kakkokari-gtyih changed the title enhance(frontend): リプライ・リノート・リアクションの総数を表示するか設定で選べるように enhance(frontend): リアクションの総数を表示するか設定で選べるように Mar 7, 2024
@syuilo syuilo merged commit 27f823e into misskey-dev:develop Mar 8, 2024
15 of 17 checks passed
@syuilo
Copy link
Member

syuilo commented Mar 8, 2024

🙏

@kakkokari-gtyih kakkokari-gtyih deleted the enh-notecount-settings-1 branch March 8, 2024 10:00
n1lsqn pushed a commit to n1lsqn/misskey that referenced this pull request Mar 10, 2024
* enhance(frontend): リプライ・リノート・リアクションの総数を表示するか設定で選べるように (MisskeyIO#512)

(cherry picked from commit 3c8475e)

* fix: いいねのみの場合は強制的にカウント表示

* make `showReactionsCount` default false

* リアクションだけ

* けしわすれ

* けしわすれ2

---------

Co-authored-by: まっちゃとーにゅ <17376330+u1-liquid@users.noreply.github.com>
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

Successfully merging this pull request may close these issues.

4 participants