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

TF-3267 Implement HTML attachment preview #3327

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

tddang-linagora
Copy link
Contributor

@tddang-linagora tddang-linagora commented Dec 6, 2024

Issue

Demo

Screen.Recording.2024-12-09.at.13.17.46.mov

Copy link

github-actions bot commented Dec 6, 2024

This PR has been deployed to https://linagora.github.io/tmail-flutter/3327.

@hoangdat
Copy link
Member

hoangdat commented Dec 6, 2024

hi @chibenwa , please take a look on the UI for HTML preview. Hope to get your feedback to improve it. Thanks

Copy link
Member

@dab246 dab246 left a comment

Choose a reason for hiding this comment

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

  • Click outside html viewer cannot close it
click-ouside-does-not-close.mov
  • Click hyper link can not open new tab
click-link-can-not-open-new-tab.mov

@tddang-linagora
Copy link
Contributor Author

Click hyper link can not open new tab

@dab246 Should we disable all hyperlink? This is a preview function, and viewing should be all user can do with it.
cc @hoangdat

@dab246
Copy link
Member

dab246 commented Dec 9, 2024

  • Missing content when preview rich html file
Screen.Recording.2024-12-09.at.16.07.29.mov

@dab246
Copy link
Member

dab246 commented Dec 9, 2024

  • Add download button on right top bar in html viewer as PDF viewer

@tddang-linagora
Copy link
Contributor Author

Click outside html viewer cannot close it

can-close-when-click-outside.mov

Click hyper link can not open new tab

can-open-external-link.mov

Missing content when preview rich html file

can-view-content.mov

Add download button on right top bar in html viewer as PDF viewer

can-download.mov

@dab246
Copy link
Member

dab246 commented Dec 11, 2024

  • Please fix size when loading html as above demo video
Screenshot 2024-12-11 at 12 51 51

@dab246
Copy link
Member

dab246 commented Dec 11, 2024

  • When click hyper link then can't open new tab

Weblate - web-based localization.html.zip

Screen.Recording.2024-12-11.at.13.42.16.mov

@dab246
Copy link
Member

dab246 commented Dec 11, 2024

  • No response when file size is large (several MB)

Twake Chat.html.zip

Screen.Recording.2024-12-11.at.14.07.07.mov

@tddang-linagora
Copy link
Contributor Author

Please fix size when loading html as above demo video
No response when file size is large (several MB)

Fix size and add loading

Preview-loading.mov

@tddang-linagora
Copy link
Contributor Author

When click hyper link then can't open new tab

Screen.Recording.2024-12-11.at.16.52.06.mov

dab246
dab246 previously approved these changes Dec 13, 2024
Copy link
Member

@dab246 dab246 left a comment

Choose a reason for hiding this comment

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

  • Please rebase it

Comment on lines +46 to +47
child: ColoredBox(
color: Colors.transparent,
Copy link
Member

Choose a reason for hiding this comment

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

IMO, we don't need it

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It is needed for GestureDetector to receive tap outside Html view

children: [
TopBarAttachmentViewer(
title: title,
closeAction: Get.back,
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
closeAction: Get.back,
closeAction: popBack,

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants