Skip to content

Show redacted messages using the standard Message widget#662

Merged
kevinaboos merged 4 commits intoproject-robius:mainfrom
kevinaboos:show_redactions_as_regular_messages
Jan 16, 2026
Merged

Show redacted messages using the standard Message widget#662
kevinaboos merged 4 commits intoproject-robius:mainfrom
kevinaboos:show_redactions_as_regular_messages

Conversation

@kevinaboos
Copy link
Member

@kevinaboos kevinaboos commented Jan 15, 2026

instead of the old approach of making it a tiny SmallStateEvent.

This is more in line with the Matrix spec and the behavior of other clients. It also allows us to use richer formatting for the "message deleted" text.

Fetch the displayable name of the redactor (the one who deleted the message) and use it in the Message widget for a redacted message, but because it's expensive we do not use it for simple text previews of a redacted message.

This PR also redesigns the user_profile_cache API to make it easier to use and also more efficient (less cloning), and also to allow disambiguating between whether a user's information came from the per-room data or profile/account data.

instead of the old approach of making it a tiny SmallStateEvent.

This is more in line with the Matrix spec and the behavior of other clients.
It also allows us to use richer formatting for the "message deleted" text.
…essage)

Also redesign the user_profile_cache API to make it easier to use
and more efficient, and also to allow disambiguating between whether
a user's information came from the per-room data or profile/account data.
Just wait for the `TimelineDetails` to be properly fetched,
which is better and way more efficient than us duplicately querying
the room member list repeatedly / in the interim.
@kevinaboos kevinaboos merged commit 044c0ab into project-robius:main Jan 16, 2026
11 checks passed
@kevinaboos kevinaboos deleted the show_redactions_as_regular_messages branch January 16, 2026 07:08
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.

1 participant