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

feat(frontend): ノートのリアクションボタンのアイコンを変更し、削除をせずにリアクションを変更できるように #11157

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

mappi-pr
Copy link
Contributor

@mappi-pr mappi-pr commented Jul 7, 2023

What

  1. 「いいねのみ」の場合のハートアイコンを、リアクションなしの場合は+付きのハート、リアクションありの場合は-付きのハートを表示するように変更。
image image
  1. リアクションボタンのアイコンを顔アイコンに変更。
image
  1. リアクション済の場合でも、顔アイコンをクリックしてリアクションピッカーを表示するように変更。
     ※変更時は変更ダイアログを出力する。
image

Why

  • リアクションボタンのアイコンを顔アイコンに変更する
    • より多くの環境でリアクションの文脈として使われているため
  • そのノートにリアクションしているかどうかに関わらず、常にリアクションを選択するUIにする

※ただし、リアクション変更時には変更ダイアログを出して、同じノートに対して複数リアクションできると勘違いされないようにする。

Additional info (optional)

ローカル環境のChrome/Firefoxで以下の項目のテスト済み

  • TLのノート(いいねのみ)で新しく❤️をつける
  • TLのノート(いいねのみ)で既存の❤️を削除する
  • ノート詳細画面(いいねのみ)で新しく❤️をつける
  • ノート詳細画面(いいねのみ)で既存の❤️を削除する
  • TLのノート(通常リアクション受入)で新しくリアクションをつける
  • TLのノート(通常リアクション受入)で別のリアクションをつける
  • TLのノート(通常リアクション受入)で既存のリアクションを削除する
  • ノート詳細画面(通常リアクション受入)で新しくリアクションをつける
  • ノート詳細画面(通常リアクション受入)で別のリアクションをつける
  • ノート詳細画面(通常リアクション受入)で既存のリアクションを削除する

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 Jul 7, 2023
@github-actions github-actions bot requested review from EbiseLutica and syuilo July 7, 2023 06:22
@codecov
Copy link

codecov bot commented Jul 7, 2023

Codecov Report

Merging #11157 (409c62b) into develop (74a05ec) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop   #11157      +/-   ##
===========================================
+ Coverage    77.52%   77.55%   +0.02%     
===========================================
  Files          908      909       +1     
  Lines        91747    91830      +83     
  Branches      7617     7618       +1     
===========================================
+ Hits         71131    71216      +85     
+ Misses       20616    20614       -2     
Impacted Files Coverage Δ
packages/frontend/src/components/MkNote.vue 100.00% <100.00%> (ø)

... and 2 files with indirect coverage changes

@acid-chicken
Copy link
Member

半分独り言ですが、内容の是非はともかくとして Storybook のストーリーが欲しくなる

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
Development

Successfully merging this pull request may close these issues.

3 participants