Paintroid-450 : Add share menu option in paint view#49
Paintroid-450 : Add share menu option in paint view#49vishad-tyagi wants to merge 6 commits intoCatrobat:developfrom
Conversation
There was a problem hiding this comment.
The share package seems to be deprecated see here. Please always check if packages are maintained before adding them.
Also- add tests please :)
|
@Lenkomotive I have changed the package to share_plus. and have added the tests also. while Native share windows cannot be tested in Flutter widget tests because these tests run in a Dart VM, that does not interact with native components. I have implemented a workaround of checking the entire widget tree before and after, looking for any unexpected behaviour and error dialog. If anything else needs to be changed let me know. |
msesko
left a comment
There was a problem hiding this comment.
pls check my comments. thanks
packages/features/workspace_screen/test/widget/workspace_screen_test.dart
Outdated
Show resolved
Hide resolved
packages/features/workspace_screen/lib/src/components/top_bar/overflow_menu.dart
Outdated
Show resolved
Hide resolved
|
Thank you for the PR! |
|
@juliajulie95 I intended to do the same initially only but was waiting for additional info like the default format of shared images, whether manual image compression is required or not etc 🙂 , I tried to get more info on the Jira ticket only. So I ended up placing a text-only |
|
Sorry for that. |
|
Are we still working on this ticket? @juliajulie95 @msesko |
Ticket
https://catrobat.atlassian.net/browse/PAINTROID-450
Paintroid-450 : Add share menu option in paint view
Description
Add "share" option to the paint view overflow menu.
This should open the phone native share window.
New Features and Enhancements
Refactorings and Bug Fixes
Checklist
Your checklist for this pull request
Please review the contributing guidelines and wiki pages of this repository.