Skip to content

TF-3644 Handle scroll thread detail when cursor is in email content area #3716

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

Conversation

dab246
Copy link
Member

@dab246 dab246 commented May 9, 2025

Issue

#3644

Root cause

HtmlElementView (iframe) does not allow Flutter to intervene at the mouse scroll level due to platform view class limitation.

Solution

Use JavaScript in iframe to forward scroll event to Flutter

  • Send scroll event from iframe to Flutter
  • Catch message event from iframe in Flutter Web by create a JS interop to listen for postMessage from iframe

Resolved

Screen.Recording.2025-05-09.at.20.53.50.online-video-cutter.com.mp4

Copy link

github-actions bot commented May 9, 2025

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

@dab246 dab246 changed the title TF-3644 POC scroll thread view TF-3644 Handle scroll thread detail when cursor is in email content area May 9, 2025
@dab246 dab246 force-pushed the feature/tf-3644-poc-scroll-thread-view branch from 2da133d to a086b12 Compare May 9, 2025 14:01
@dab246 dab246 force-pushed the feature/tf-3644-poc-scroll-thread-view branch from a086b12 to 4900e4f Compare May 20, 2025 04:44
@dab246 dab246 changed the base branch from feature/TF-3642-Thread-Detail-view-multiple-emails to feature/thread-detail-part-2-3-4 May 20, 2025 04:45
@dab246 dab246 requested a review from hoangdat May 20, 2025 11:11
@dab246 dab246 force-pushed the feature/tf-3644-poc-scroll-thread-view branch from cd0333b to 8909cd5 Compare May 21, 2025 08:31
@hoangdat hoangdat merged commit 3284e05 into feature/thread-detail-part-2-3-4 May 23, 2025
19 checks passed
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.

3 participants