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: add reply to message functionality #561

Merged
merged 10 commits into from
May 6, 2024

Conversation

Spiral-Memory
Copy link
Collaborator

@Spiral-Memory Spiral-Memory commented Apr 12, 2024

Brief Title

This PR adds a feature to quote message

Acceptance Criteria fulfillment

  • added quote icon to message toolbox
  • added global state to keep track of quote messages
  • added UI preview of quoted message
  • added support for attachment quoting

Fixes #560

Video/Screenshots

2024-04-12.23-48-04.mp4

@Spiral-Memory Spiral-Memory marked this pull request as draft April 12, 2024 09:18
@Spiral-Memory Spiral-Memory force-pushed the feat/quote-msg branch 5 times, most recently from 78ee12b to 36bcec4 Compare April 12, 2024 12:27
@Spiral-Memory Spiral-Memory marked this pull request as ready for review April 12, 2024 18:08
@Spiral-Memory
Copy link
Collaborator Author

Spiral-Memory commented Apr 12, 2024

Hey @sidmohanty11 , currently, I've added support for quoting a single message in EC. In RC, multiple messages can also be quoted, but I'm not adding that feature for now, as we typically reply to a single message. It can be added later on if necessary. Currently, I don't think that will be a valuable addition. What's your opinion on this?

Also, while quoting attachments, in RC, it shows image preview and download button, which I feel is unnecessary considering an in-chat app.

Please do let me know your thoughts. If required, I will add those or later if required, it can be added.

Additionally, the UI for when recursive messages are quoted is being handled for both the normal UI and for this addition as well in Issue #563 by Akshun.

@Spiral-Memory Spiral-Memory changed the title feat: add quote message functionality feat: add reply to message functionality Apr 12, 2024
@sidmohanty11 sidmohanty11 merged commit 7ffb61b into RocketChat:develop May 6, 2024
3 checks passed
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.

[FEAT]: Option to reply to a message
2 participants