Skip to content
This repository was archived by the owner on Jul 11, 2025. It is now read-only.

feat(reactions): add tap-to-remove for own reactions in drawer#1595

Merged
thierryskoda merged 4 commits intomainfrom
lr/tap-remove-own-reaction
Feb 10, 2025
Merged

feat(reactions): add tap-to-remove for own reactions in drawer#1595
thierryskoda merged 4 commits intomainfrom
lr/tap-remove-own-reaction

Conversation

@lourou
Copy link
Copy Markdown
Member

@lourou lourou commented Feb 6, 2025

Changes:

  • Added tap-to-remove functionality for the user's own reactions in the reactions drawer
  • Implemented subtle visual feedback using opacity change on press
  • Preserved existing reaction sorting and filtering behavior

Fixes: #1592

@lourou lourou requested a review from a team as a code owner February 6, 2025 16:10
@thierryskoda thierryskoda requested a review from a team as a code owner February 10, 2025 15:50
lourou and others added 2 commits February 10, 2025 10:50
- Add ability to remove reactions by tapping them in the reactions drawer
- Only own reactions are tappable with subtle opacity feedback (0.75)
- Keep reactions list sorted with own reactions on top
- Maintain existing filtering and sorting behavior
@thierryskoda thierryskoda force-pushed the lr/tap-remove-own-reaction branch from abc8969 to 6aee21d Compare February 10, 2025 15:50
@thierryskoda
Copy link
Copy Markdown
Contributor

@lourou modified some code. Here's Loom video of it https://www.loom.com/share/9133cdb69fec4771b68e84a41d63f5f0

@thierryskoda thierryskoda merged commit 4465d64 into main Feb 10, 2025
@thierryskoda thierryskoda deleted the lr/tap-remove-own-reaction branch February 10, 2025 16:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: In drawer, implement “tap to remove” my own reaction

2 participants