Skip to content
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

TW-616 List all image & video of Chat in Chat details #650

Merged
merged 5 commits into from
Sep 21, 2023

Conversation

drminh2807
Copy link
Contributor

@drminh2807 drminh2807 commented Sep 18, 2023

Depend on

Issue

Demo

Screen.Recording.2023-09-18.at.10.45.09.mov
RPReplay_Final1695008083.MP4

Image viewer

Screen.Recording.2023-09-19.at.09.49.13.mov

@drminh2807 drminh2807 temporarily deployed to PR-650 September 18, 2023 04:00 — with GitHub Actions Inactive
@drminh2807 drminh2807 force-pushed the TW-616-List-all-image-video-of-Chat-in-Chat-details branch from cf55859 to 808ba61 Compare September 18, 2023 04:07
@drminh2807 drminh2807 temporarily deployed to PR-650 September 18, 2023 04:07 — with GitHub Actions Inactive
@github-actions
Copy link

This PR has been deployed to https://linagora.github.io/twake-on-matrix/650

@sherlockvn
Copy link
Contributor

sherlockvn commented Sep 18, 2023

Issue

Demo

Screen.Recording.2023-09-18.at.10.45.09.mov
RPReplay_Final1695008083.MP4

are images and videos are thumbnails or real files, because it looks slow when loading. And please demo the case when you click to image or videos

lib/widgets/mxc_image.dart Outdated Show resolved Hide resolved
@drminh2807 drminh2807 force-pushed the TW-616-List-all-image-video-of-Chat-in-Chat-details branch from 808ba61 to b4e319c Compare September 19, 2023 02:08
@drminh2807 drminh2807 temporarily deployed to PR-650 September 19, 2023 02:08 — with GitHub Actions Inactive
@drminh2807
Copy link
Contributor Author

are images and videos are thumbnails or real files, because it looks slow when loading. And please demo the case when you click to image or videos

@sherlockvn it is thumbnail but need load many thumbnails so it will take time, but it just happen on first load, because then it is cached
I also updated demo for image preview, video isn't have video player

@drminh2807 drminh2807 force-pushed the TW-616-List-all-image-video-of-Chat-in-Chat-details branch from b4e319c to c1b7e05 Compare September 19, 2023 03:43
@drminh2807 drminh2807 temporarily deployed to PR-650 September 19, 2023 03:43 — with GitHub Actions Inactive
@drminh2807 drminh2807 force-pushed the TW-616-List-all-image-video-of-Chat-in-Chat-details branch from c1b7e05 to b9bff45 Compare September 19, 2023 03:56
@drminh2807 drminh2807 temporarily deployed to PR-650 September 19, 2023 03:56 — with GitHub Actions Inactive
@drminh2807 drminh2807 force-pushed the TW-616-List-all-image-video-of-Chat-in-Chat-details branch from b9bff45 to 4b59f65 Compare September 20, 2023 04:49
@drminh2807 drminh2807 temporarily deployed to PR-650 September 20, 2023 04:49 — with GitHub Actions Inactive
@drminh2807 drminh2807 temporarily deployed to PR-650 September 21, 2023 03:32 — with GitHub Actions Inactive
@drminh2807 drminh2807 temporarily deployed to PR-650 September 21, 2023 03:49 — with GitHub Actions Inactive
@drminh2807 drminh2807 temporarily deployed to PR-650 September 21, 2023 06:43 — with GitHub Actions Inactive
@drminh2807 drminh2807 temporarily deployed to PR-650 September 21, 2023 07:03 — with GitHub Actions Inactive
@drminh2807 drminh2807 force-pushed the TW-616-List-all-image-video-of-Chat-in-Chat-details branch from 6dbe17b to 85c66c9 Compare September 21, 2023 08:37
@drminh2807 drminh2807 force-pushed the TW-616-List-all-image-video-of-Chat-in-Chat-details branch from 85c66c9 to d644877 Compare September 21, 2023 08:41
@hoangdat hoangdat merged commit 0203c4c into main Sep 21, 2023
3 checks passed
@hoangdat hoangdat deleted the TW-616-List-all-image-video-of-Chat-in-Chat-details branch November 13, 2024 04:03
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.

5 participants