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(reactions): add tap-to-remove for own reactions in drawer #1595

Merged
merged 4 commits into from
Feb 10, 2025

Conversation

lourou
Copy link
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
Collaborator

@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
4 of 6 checks passed
@thierryskoda thierryskoda deleted the lr/tap-remove-own-reaction branch February 10, 2025 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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