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

Issue #67 Add Additional Sharing Options #87

Merged
merged 2 commits into from
Sep 14, 2024

Conversation

vverma022
Copy link
Contributor

Added Additional Sharing Options

##Description:
This pull request introduces additional sharing options for the application. Users can now share content directly to WhatsApp, Twitter, and Instagram. The implementation includes:

WhatsApp: Redirects to WhatsApp with the content link.

  1. Twitter: Opens a tweet compose window with the content link.

  2. Instagram: Copies the link to the clipboard for manual sharing (as direct URL sharing is not supported).

##Changes:

  1. Updated handleShare function to handle new sharing options.

  2. Added buttons for WhatsApp, Twitter, and Instagram in the sharing dialog.

##Testing:

  1. Verified that sharing to WhatsApp and Twitter works as expected.

  2. Confirmed that the Instagram sharing option correctly copies the link to the clipboard.

Closes #67
Screenshot 2024-09-12 at 7 56 04 PM

@SujithThirumalaisamy
Copy link
Collaborator

Can we have a dropdown for this? Dialog seems a bit disctracting. Dropdown will have a better UX

@vverma022
Copy link
Contributor Author

Alright I convert it to a drop-down.

@vverma022
Copy link
Contributor Author

Can we have a dropdown for this? Dialog seems a bit disctracting. Dropdown will have a better UX

Screenshot 2024-09-14 at 1 34 36 AM

@SujithThirumalaisamy
Copy link
Collaborator

Thankyou sir. Great work

@SujithThirumalaisamy SujithThirumalaisamy merged commit d5c1bb8 into code100x:main Sep 14, 2024
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Additional Sharing Options
2 participants