-
Notifications
You must be signed in to change notification settings - Fork 57
Add the ability to share downloaded attachments. #205
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
base: master
Are you sure you want to change the base?
Conversation
@Dima-Android could you please have a look at this when you get the time? Shouldn't be too much of a major change, small QoL improvement. |
Thanks for the contribution! Initially, we'll want to mirror the iOS functionality: a share button in the reader that opens a menu with strings (@Dima-Android, can you reimport iOS strings? The current file only has an outdated "Export" option.) This will probably make more sense in a separate PR. As for the changes in this PR:
So let's hold off on this PR for now. But if you're open to working on a PR for the reader toolbar, we'd happily accept that. @Dima-Android might be able to provide pointers for determining whether a PDF has annotations and how to get an annotated copy. |
Thanks for the feedback. I'll address your comments one by one:
To elaborate on (2): Suppose I have an article with both the open-access PDF and the submitted arXiv PDF, both of which are download locally. Sharing from the Items view (instead of opening and viewing details) allows me to share both files at once. I added this to more easily export entire items (as opposed to a single Attachment) from Zotero, something I have watched professors wish for. In particular, their usecase is sharing a research paper (often part of course material), with their own notes and explanations on the matter, with ease. Screenshots are attached below. |
Refactor sharing into AttachmentFileShareController. Minor fixups.
b76feff
to
b893439
Compare
Here's how I plan to proceed:
Is there any dedicated channel (like Discord, IRC, etc.) for development related communication? |
Looking at:
This PR adds the ability to share downloaded attachment files (and technically text URLs, though this is unused) with other apps. Currently, one must long press the attachment in the Item Details view, or one can share all downloaded attachments for a library item by long pressing the item. More "findable" options can be implemented as desired.
This PR also rewords the AllItems "Remove Download" option to be more clear.
Screenshots:
