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

Share improvements #202

Closed
wants to merge 5 commits into from
Closed

Conversation

rsammelson
Copy link
Contributor

@rsammelson rsammelson commented Jul 3, 2023

Fixes #183 and adds modal to select what the user would like to share (link to post or link to post content). Fixes #255. Also should fix #474.

share-modal

@rsammelson
Copy link
Contributor Author

I'm guessing I should move the toast to the bottom. I didn't like how it covered the bottom bar, but now I realize that is a general issue that needs to be solved (#147).

@rsammelson rsammelson force-pushed the share-improvements branch from aebc7a0 to 9cac9fa Compare July 4, 2023 01:37
@rsammelson rsammelson marked this pull request as ready for review July 4, 2023 01:39
@rsammelson rsammelson force-pushed the share-improvements branch 2 times, most recently from fe14238 to 81f685b Compare July 4, 2023 03:32
@rsammelson
Copy link
Contributor Author

@aeharding This is back to being ready to merge.

@rsammelson
Copy link
Contributor Author

@aeharding, sorry for the pings, but are you ready for this PR if I fix merge conflicts, or do you want to wait?

@rsammelson rsammelson force-pushed the share-improvements branch from 81f685b to 3280a5c Compare July 6, 2023 00:31
@rsammelson rsammelson marked this pull request as draft July 6, 2023 06:05
@rsammelson rsammelson force-pushed the share-improvements branch from 3280a5c to 2fc020d Compare July 6, 2023 06:16
@rsammelson rsammelson marked this pull request as ready for review July 6, 2023 06:16
@rsammelson
Copy link
Contributor Author

@aeharding I don't think this depends on anything else now, and I have it working with the image gallery share button too.

@aeharding
Copy link
Owner

I'll take a look soon. To be honest, I'm skeptical we need to prompt the user to ask them what they would like to share 🤔 but I will try it out before any really feedback :)

@rsammelson
Copy link
Contributor Author

Oh, I like being able to share content or the post

@rsammelson rsammelson force-pushed the share-improvements branch 3 times, most recently from 0506fc4 to 207e156 Compare July 11, 2023 03:29
@aeharding
Copy link
Owner

Oh, I like being able to share content or the post

Was this a feature in Apollo? I know there was the ability to share a picture of the post, but that would be a separate feature.

@rsammelson
Copy link
Contributor Author

I'm not sure, I never used Apollo. I just want to be able to share the post itself with people so they can interact with it (comment/vote). The app I used had a similar feature.

I assume most people aren't sharing often enough to be bothered by a two option popup when they click share, but I could add a setting to always default to sharing the media link if you want. I would really like to get this one merged so I can use it easily.

@Penagwin
Copy link
Contributor

Penagwin commented Jul 11, 2023

Here's a screenshot of the context menu from Apollo - I don't remember if anything was below report.

@aeharding Do you want to combine all of the sharing and saving related meta-discussion with #69? There's a several related moving parts with them.

image

@rsammelson
Copy link
Contributor Author

In a future PR, I could add more share options to the modal created here to at least allow non-iOS users to be able to share media directly and also a share as image option. Personally I think it would be a better user experience if they only show up when clicking share, the post actions menu is already very long, and it standardizes the experience between using the share button and post actions (and I can add share as a swipe action with the same component along with customizable swipes).

@Penagwin
Copy link
Contributor

What I'm wondering is if we replace the "tap-hold" that iOS 16 users have with just this screen. Then it would be consistent across all platforms?

@rsammelson
Copy link
Contributor Author

Since tap and hold is a often available OS-level thing, I think leaving it is best to match other iOS apps. But I don't feel strongly about that.

@rsammelson rsammelson force-pushed the share-improvements branch from 207e156 to 298211f Compare July 13, 2023 01:06
@rsammelson rsammelson force-pushed the share-improvements branch 3 times, most recently from 3c5aab3 to 007fa2f Compare July 20, 2023 03:51
Currently the options are to share a link to the post or the link
content of the post if it exists (if it does not the modal is not used).
@rsammelson rsammelson force-pushed the share-improvements branch from b05c33b to 706d7e8 Compare July 28, 2023 02:04
@rsammelson
Copy link
Contributor Author

@aeharding I added share as a swipe option and updated from the latest main.

@aeharding
Copy link
Owner

Thanks. I still need time to look at this

@aeharding aeharding force-pushed the main branch 2 times, most recently from 4346eb6 to 86d9319 Compare August 5, 2023 06:12
@aeharding
Copy link
Owner

Thanks for the PR! I am closing this because it has become quite stale. Additionally, there is now an option to share the post image from the media viewer :)

@aeharding aeharding closed this Dec 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants