Skip to content

fix(chat): do not separate edited messages from the group - #19006

Open
Antreesy wants to merge 3 commits into
mainfrom
fix/noid/edited-combine
Open

fix(chat): do not separate edited messages from the group#19006
Antreesy wants to merge 3 commits into
mainfrom
fix/noid/edited-combine

Conversation

@Antreesy

@Antreesy Antreesy commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

☑️ Resolves

  • Behaviour change for edited messages:
    • Edit marker is unified (pencil icon + avatar if differs from author, details in message actions menu)
    • Messages no longer ungrouped, if edited

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

🖌️ UI Checklist

🖼️ Screenshots / Screencasts

🏚️ Before 🏡 After
Edited caption is ungrouped, all edited messages ungrouped, foreign edits are not visible Edited caption part of combined message, all edited messages are part of respective group, foreign edits highlighted with avatar
image image
image image

🏁 Checklist

  • 🌏 Tested with different browsers / clients:
    • Chromium (Chrome / Edge / Opera / Brave)
    • Firefox
    • Safari
    • Talk Desktop
    • Integrations with Files sidebar and other apps
    • Not risky to browser differences / client
  • 🖌️ Design was reviewed, approved or inspired by the design team
  • ⛑️ Tests are included or not possible
  • 📗 User documentation in https://github.com/nextcloud/documentation/tree/master/user_manual/talk has been updated or is not required

@Antreesy
fix(chat): do not separate edited messages from the group
fa332a2 
Assisted-by: ClaudeCode:claude-sonnet-5
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
@Antreesy
fix(chat): fix condition for showing editor avatar
0541f10 
- if both are users, avatar wasn't shown

Assisted-by: ClaudeCode:claude-sonnet-5
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
@Antreesy
fix(chat): show pencil icon for all messages
83f49e3 
- including list view
- remove duplicate string next to display name

Assisted-by: ClaudeCode:claude-sonnet-5
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
@Antreesy Antreesy added this to the ⛅ Next Beta/RC (35) milestone Aug 14, 2026
@Antreesy Antreesy self-assigned this Aug 14, 2026
@nextcloud-command nextcloud-command added the AI assisted This PR contains AI-assisted commits label Aug 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nimishavijay nimishavijay Awaiting requested review from nimishavijay
@DorraJaouad DorraJaouad Awaiting requested review from DorraJaouad

At least 1 approving review is required to merge this pull request.

Assignees

@Antreesy Antreesy

Labels

AI assisted This PR contains AI-assisted commits bug design feature: chat 💬 Chat and system messages feature: frontend 🖌️ "Web UI" client

Projects

None yet

Milestone

⛅ Next Beta/RC (35)

Development

Successfully merging this pull request may close these issues.

2 participants

@Antreesy @nextcloud-command