Skip to content

feat: show the message that is being deleted - #1320

Merged
Dadadah merged 2 commits into
stoatchat:mainfrom
amycatgirl:feat/show-message-to-delete-in-modal
Jul 15, 2026
Merged

feat: show the message that is being deleted#1320
Dadadah merged 2 commits into
stoatchat:mainfrom
amycatgirl:feat/show-message-to-delete-in-modal

Conversation

@amycatgirl

Copy link
Copy Markdown
Contributor

The delete message modal did not show which message was being deleted, which could have lead confusion to users trying to delete messages and accidentally pressing the wrong one. Now, the modal shows which message is being deleted, causing less confusion.

How was this PR tested?

  • Tried deleting my own message
    • Check if embedded content (attachments and link previews) appears correctly
    • Check if replies are also shown alongside the message
  • Tried deleting other person's message
    • Check if embedded content (attachments and link previews) appears correctly
    • Check if replies are also shown alongside the message

Screenshots & Screencasts (if appropriate)

image image image

Checklist:

  • I have carefully read the contributing guidelines
  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation if applicable
  • I have no unrelated changes in the PR
  • I have confirmed that any new dependencies are strictly necessary
  • I have written tests for new code (if applicable)
  • I have followed naming conventions/patterns in the surrounding code

Please declare, if any, LLM usage involved in creating this PR

No AI was used at the time of writing the pull request.

Signed-off-by: Amelia <amy+git@amogus.cloud>
Closes stoatchat#1269

Signed-off-by: Amelia <amy+git@amogus.cloud>
@amycatgirl
amycatgirl requested review from a team as code owners July 1, 2026 15:21
@Dadadah Dadadah linked an issue Jul 10, 2026 that may be closed by this pull request
@Dadadah
Dadadah merged commit fa1a99f into stoatchat:main Jul 15, 2026
9 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.

Delete Message Preview

3 participants