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

📎 Add more sources to upload attachments #289

Closed
jakobroehrl opened this issue Mar 3, 2020 · 17 comments
Closed

📎 Add more sources to upload attachments #289

jakobroehrl opened this issue Mar 3, 2020 · 17 comments
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@jakobroehrl
Copy link

jakobroehrl commented Mar 3, 2020

Is your feature request related to a problem? Please describe.
new button "camara" within the attachment section to make a photo and upload it directly as new attachment

Describe alternatives you've considered
Start the camara App manually, take a pic, open deck go to attachments, click the upload Button, search for the photo, add it

@jakobroehrl jakobroehrl added the enhancement New feature or request label Mar 3, 2020
@desperateCoder
Copy link
Collaborator

Yep, that's a good one!

@stefan-niedermann
Copy link
Owner

stefan-niedermann commented Mar 3, 2020

Sure, makes sense. I guess we will have either an expandable floating action button or a bottom sheet like in whatsapp/telegram in the future.

@stefan-niedermann stefan-niedermann changed the title direct photo upload as an attachment 📎 Add more sources to upload attachments Mar 3, 2020
@stefan-niedermann stefan-niedermann added the help wanted Extra attention is needed label Apr 24, 2020
@stefan-niedermann
Copy link
Owner

I have some troubles when receiving files which are not physically on the device yet.

I guess we have to store a local copy of the files in the app-directory and upload it from there, but i do not yet fully understand the concept of the DocumentProvider. Knowledge input or a Pull Request is welcome 🙂.

@q-wertz
Copy link

q-wertz commented Apr 25, 2020

Maybe this is also related nextcloud/deck#564 and nextcloud/server#10292

stefan-niedermann added a commit that referenced this issue Oct 29, 2020
Make contacts upload work

Signed-off-by: Stefan Niedermann <info@niedermann.it>
stefan-niedermann added a commit that referenced this issue Oct 29, 2020
Minor clean up

Signed-off-by: Stefan Niedermann <info@niedermann.it>
stefan-niedermann added a commit that referenced this issue Oct 29, 2020
Fix contact picker

Signed-off-by: Stefan Niedermann <info@niedermann.it>
stefan-niedermann added a commit that referenced this issue Oct 29, 2020
Design fine tuning

Signed-off-by: Stefan Niedermann <info@niedermann.it>
stefan-niedermann added a commit that referenced this issue Oct 29, 2020
Design fine tuning

Signed-off-by: Stefan Niedermann <info@niedermann.it>
stefan-niedermann added a commit that referenced this issue Oct 29, 2020
Design fine tuning

Signed-off-by: Stefan Niedermann <info@niedermann.it>
@stefan-niedermann
Copy link
Owner

stefan-niedermann commented Nov 2, 2020

@jakobroehrl @q-wertz @splitt3r please have a look at this PR - It would be awesome if you could help testing the new stuff 🚀 🙂

@stefan-niedermann
Copy link
Owner

@jakobroehrl @q-wertz @splitt3r since you were interested in this issue, i'd like to ask you whether you are interested in testing some more improvements we've done before i release it as a beta? 🙂

Here is the APK to download and test - feedback is very welcome here.

@jakobroehrl
Copy link
Author

wow! This is working awesome!

@q-wertz
Copy link

q-wertz commented Nov 10, 2020

Looks amazing 🎉
My only problem was that on first sight I had trouble to find the camera. Might be more intuitive to not have it in the Gallery tab but in a separate one with own symbol?
Screenshot_20201110-134918~3

@stefan-niedermann
Copy link
Owner

Hmm, did the upper left item (with the camera symbol) not display a live preview of the camera for you? (because it is black but it should display a live cam)

I orientated pretty much at the attachments picker of Telegram and it seemed to be pretty intuitive (for me at least 😆 )

@q-wertz
Copy link

q-wertz commented Nov 10, 2020

It is black because it was covered 😆
So everything fine. Maybe just also had the wrong expectations from the previous version/other apps. 👍

@stefan-niedermann
Copy link
Owner

According to the material guidelines the bottom navigation (which we use) should only host up to 5 items - i planned to add one for audio one for video.

The concept idea is to have one picker per media type and each of the pickers shows a "primary / native" action + previews for quick access.

  • camera + images
  • native contact picker + direct contacts
  • native file picker + downloaded files

and maybe in the future something like

  • audio recorder + audio files
  • video capturing + videos

But we for sure can still discuss this, maybe you have screenshots of some apps to compare with?

@stefan-niedermann
Copy link
Owner

The current state will be in version 1.12.0 (which is already tagged for F-Droid and available on Play Store beta channel).

Feel free to still discuss this feature in this or a new issue, but it "works as expected" and noone reported any bugs so far, so i thought: "let's get some more feedback from the community" 🙂

@aproposnix
Copy link

I don't mean to seem so pedantic but the rounded corners don't feel right when the menu is fully loaded with the round corners. Other than that, this is an awesome new feature.

signal-2020-11-10-223042

@stefan-niedermann
Copy link
Owner

Indeed, i will have a look whether we can animate those corners :).

@q-wertz
Copy link

q-wertz commented Nov 12, 2020

According to the material guidelines the bottom navigation (which we use) should only host up to 5 items - i planned to add one for audio one for video.

The concept idea is to have one picker per media type and each of the pickers shows a "primary / native" action + previews for quick access.

* camera + images

* native contact picker + direct contacts

* native file picker + downloaded files

and maybe in the future something like

* audio recorder + audio files

* video capturing + videos

But we for sure can still discuss this, maybe you have screenshots of some apps to compare with?

I think it makes sense to design it the way you said:+1:

Nevertheless, here are some screenshots how different messengers handle this (if you need some inspiration one day and maybe you don't have one of these 😉):
Threema
Whatsapp
Element
Slack

@jakobroehrl
Copy link
Author

@stefan-niedermann
This is so awesome implemented.
Is there a change to implement this function in Android-talk, too?

@stefan-niedermann
Copy link
Owner

I am not aware of one, but i happily will have a talk with @AndyScherzinger and @tobiasKaminsky if they are interested to adapt the implementation 😉 It should be pretty portable and easy to adapt I think...

Merry christmas and stay well!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

5 participants