-
Notifications
You must be signed in to change notification settings - Fork 72
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
#2 Cherry pick add style/attachment/inline image
button to app bar composer on mobile (#2532, #2584)
#2794
#2 Cherry pick add style/attachment/inline image
button to app bar composer on mobile (#2532, #2584)
#2794
Conversation
attachment image
button to app bar composer (#2532)attachment/image
button to app bar composer (#2532)
attachment/image
button to app bar composer (#2532)attachment/image
button to app bar composer (#2532)
This PR has been deployed to https://linagora.github.io/tmail-flutter/2794. |
attachment/image
button to app bar composer (#2532)style/attachment/inline image
button to app bar composer on mobile (#2532)
style/attachment/inline image
button to app bar composer on mobile (#2532)style/attachment/inline image
button to app bar composer on mobile (#2532, #2584)
9865b0c
to
e6eff37
Compare
Rebase to the |
|
Root cause
@hoangdat Please fetch latest PR then test again DemoScreen_recording_20240620_162644.webm |
|
linagora/enough_html_editor#30 was merged. Please update pubspec. @dab246 |
… file Signed-off-by: dab246 <[email protected]> (cherry picked from commit 181e1ec)
… of composer in mobile
e6eff37
to
0a1d3de
Compare
Updated. Please fetch latest commit and test again. Thanks |
final uploadFileState = _uploadingStateInlineFiles.getUploadFileStateById(success.uploadId); | ||
log('UploadController::_handleProgressUploadInlineImageStateStream:uploadId: ${uploadFileState?.uploadTaskId} | fromFileShared: ${uploadFileState?.file?.isShared}'); | ||
|
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.
why we need to remove it?
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.
final uploadFileState
is just for logging purposes, we don't need it. Avoid wasting memory
|
Issue
#2532
#2584
Dependent
bugfix/add-request-focus-node-in-html
(PR #27) enough_html_editor#30Demo
mobile-app.webm
demo.mp4
demo2.mov
web-appp.online-video-cutter.com.mp4