-
Notifications
You must be signed in to change notification settings - Fork 232
Implement MSC2530 #2522
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
Implement MSC2530 #2522
Conversation
Signed-off-by: Marco Antonio Alvarez <[email protected]>
Thank you for your contribution! Here are a few things to check in the PR to ensure it's reviewed as quickly as possible:
|
Not convinced about how that column looks like, but it's sunday so I'll keep trying later |
Signed-off-by: Marco Antonio Alvarez <[email protected]>
Signed-off-by: Marco Antonio Alvarez <[email protected]>
added video captions Signed-off-by: Marco Antonio Alvarez <[email protected]>
Signed-off-by: Marco Antonio Alvarez <[email protected]>
Signed-off-by: Marco Antonio Alvarez <[email protected]>
Signed-off-by: Marco Antonio Alvarez <[email protected]>
Signed-off-by: Marco Antonio Alvarez <[email protected]>
The 'Record Screenshots' action seems be failing but I don't see any error related to this change on its log 🤔 |
Thanks for the changes! I'll merge it to a branch of mine where I can tweak these changes a bit so they fit a bit better our layout. |
5b2a45f
into
element-hq:feature/add-caption-to-image-and-video-events
Depends on matrix-org/matrix-rust-sdk#3171
Fixes #2521
Type of change
Content
Show body as image caption, per MSC 2530
Motivation and context
Caption as a different message doesn't look this good
Screenshots / GIFs
Tests
Tested devices
Checklist