-
Notifications
You must be signed in to change notification settings - Fork 85
fixes for mms attachment view/share is suboptimal (#242) and other small fixups #254
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
fixes for mms attachment view/share is suboptimal (#242) and other small fixups #254
Conversation
to media store when sharing. fixed using wrong attachment class for vcards (use lowercase on both sides of attachment mime type comparison) added share button to audio attachment preview removed old fileprovider that was never used added content provider to provide mms attachments to external activities using a filename - not mms part id - so they can be saved externally with a proper name fix bug when re-using attachment viewholder that has previously had text fixed bug of image attachments always being named 'image' rather than their filename added share button to generic file attachment type in compose message list and removed onclick handler for viewing
I'm a bit torn on the UX. downloading sounds like it would be used more often than sharing, and both aren't immediate actions. should those be in the media viewer? should we add a vcard preview? 🤔 having the share button there for every media feels like it would get annoying and not used that often. |
here's a bit of a random thought dump... for me, share/download ratio of attachments is 99% share, 1% download. i have no clue what the typical use case is though. i don't like the new share ux either. but, there's no selection of an individual attachment so it's difficult to add share/download options other than with the attachment itself (ie can't be on menu). except images and videos which already have share and download on the built-in media player activity menu... ... but audio, vcards and all other file types don't use the built-in media player so can't take advantage of the share and download options there. i really like the ability to share/download from the compose screen rather than having to go through yet another screen to get to those options. right now (with this pr)
for sure, a consistent ui for share and download is best. if a person has a file manager then sharing is kinda the same as downloading, but better, since one can just share to the file manager and put the file wherever one chooses. unfortunately devices don't tend to have a built in file manager. vcard preview; probably nice to have. some might find it useful to be able to copy data from, or dial a number from, a vcard without having to add the vcard to their contacts. 2 ideas for a consistent share and download ui for all attachment types;
more edit: another idea... build a new activity in the app named 'quik save' that accepts any shared files. user flow; |
What a mess 😆 I propose something simple like in Beeper and Signal, a long press on the message/file open the menu and you see all the option. |
@lascapi thanks for the feedback. atm long press selects message so we'd have to come up with an alternative for message selection. and it would need to be the same on the main conversation screen too, which already has swipe and short touch actions taken for other things. |
all these gremlins in the UX. this smells like it needs a rethink instead of duct tape.... 🤔 |
what if: long-press message row (but not one of the below): select message tap message row: nothing media preview has kebab menu top right to show the same media context menu media context menu includes download, share, forward, open with. (and any other options that may be necessary) |
I have suggestions but it seems a bit over complicated Double tap massage row: chosen action |
although ugly ui i think this pr should go into 4.0.10-beta. otherwise the next release will have no way to share/download audio and document (ie not image, video, vcard) attachments. should we open a new issue (pointing back here) to continue and widen the ui discussion to general ui principals that can be consistent across the entire app? ie the double-tap-edit-box-to-start-stt on the compose activity is horrible too. |
Then I'd recommend you or octo to open the issue (or another member/contributer) as most of us don't really know the technical aspects of the app |
adding this in, but leaving breadcrumbs for future readers: #266 I see you, DenverCoder9. |
changed to not save message attachments to media store when sharing.

fixed using wrong attachment class for vcards (use lowercase on both sides of attachment mime type comparison)
added share button to audio attachment preview.
removed old fileprovider that was never used.

added content provider to provide mms attachments to external activities using a filename - not mms part id - so they can be saved externally with a proper name.
fix bug when re-using attachment viewholder that has previously had text.
fixed bug of image attachments always being named 'image' rather than their filename.
added share button to generic file attachment type in compose message list and removed onclick handler for viewing.
closes #242