Skip to content

Conversation

@r10s
Copy link
Contributor

@r10s r10s commented Oct 31, 2025

this PR forces webm to be sent as file and adds some reasoning to the comment.

one can now rant a lot about apple, but this is the reality.

we tried to support webm by additional libraries, but gave up, it was a too heavy lib, that would come with all kind of security issues. also, it seems not worth the effort, afaik, webm does not come to avg user by the normal flow, only by eg. collecting things from the browser or so, dragging it to the deltachat window - but at that point, there are many formats not supported.
with .heic being sent as files we have a similar situation, btw.

one can argue that it is also unexpected that a file is not shown in-app, or gets sorted to "Files", sure, but we cannot avoid that reasonably.

and it is even more unexpected that it is shown on some and not shown on other UI. as happening a lot when one user sends a lot of webm stickers, assuming they are show to other as well. and there is not much the receiver can do about that. so consistently sending as "File" is the better tradeoff

@r10s r10s requested review from Hocuri and link2xt October 31, 2025 13:02
@r10s r10s added the bug Something is not working label Oct 31, 2025
Copy link
Collaborator

@link2xt link2xt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Put something like feat or fix in front so it does not blow up the changelog

@r10s r10s changed the title send webm as file, it is not supported by all UI fix: send webm as file, it is not supported by all UI Oct 31, 2025
@r10s r10s force-pushed the r10s/steamline-video-support branch from 676344d to 2b108f1 Compare October 31, 2025 17:50
@r10s
Copy link
Contributor Author

r10s commented Oct 31, 2025

ftr, also whatsapp and telegram seem not to support webm fully (telegram sends it as "file", whatsapp does not allow dropping it to the window at all)

cmp deltachat/deltachat-android#3980

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something is not working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants