Conversation
Removed the warning level as both Android and iOS implemented the native parts: coi-dev/flutter-deltachat-core@b88f4eb |
Re-added the warning label as it seems that some file formats are not well handled (no preview is generated). @florianhaar please update this ticket as soon as you have more information. If all issues are fixed please remove the warning label and ping me. |
As the core is already merged, but this PR needs some adjustments, I added 4092eb8 as temporary fix. This fix should get removed as soon as this PR is merged. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I somehow have the feeling the video thumbnail logic should be in the BloC and not the UI.
No change request, just some comments and potentially stuff for a followup ticket. |
f20bd8a
to
da49c93
Compare
Moved the loading logic to the MessageAttachmentBloc |
Link to the given issue
#249
Describe what the problem was / what the new feature is
Videos were shown as generic attachments and it was not possible to see what a video is.
Describe your solution
Added a method to create a thumbnail of the video and show the duration on the image. A play icon shows that the image is a video.
Additional context
Add method to set and get the duration. flutter-deltachat-core#63