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 sharing option in image carousel view #2629

Conversation

scoder1747
Copy link
Contributor

@scoder1747 scoder1747 commented Oct 29, 2024

Summary

This PR adds an ellipsis (share button) to enable sharing in the full-screen carousel viewer. Tapping the button brings up a share sheet, allowing sharing via messaging and social media apps.

r12.mov

Updated - Sharing an actual image:

Screenshot 2024-10-23 at 11 11 26 AM

Checklist

  • I have read (or I am familiar with) the Contribution Guidelines
  • I have tested the changes in this PR
  • My PR is either small, or I have split it into smaller logical commits that are easier to review
  • I have added the signoff line to all my commits. See Signing off your work
  • I have added appropriate changelog entries for the changes in this PR. See Adding changelog entries
    • I do not need to add a changelog entry. Reason: [Please provide a reason]
  • I have added appropriate Closes: or Fixes: tags in the commit messages wherever applicable, or made sure those are not needed. See Submitting patches

The ellipsis (share button) allows you to share the current image being displayed in the full screen carousel viewer.

Changelog-Changes: Add share button in Full screen image carousel view (for Image only)
Signed-off-by: Swift Coder  <[email protected]>
@scoder1747 scoder1747 changed the title Sharing option in Full screen image carousel view Add sharing option in image carousel view Oct 29, 2024
@danieldaquino danieldaquino self-requested a review November 2, 2024 22:06
The ellipsis (share button) allows you to share actual image from the full screen carousel viewer in timeline.

Changelog-Changes: Share for actual image from Carousel Viewer
Signed-off-by: Swift Coder  <[email protected]>
@scoder1747 scoder1747 force-pushed the add-share-button-in-full-screen-image-carousel branch from 218b4c3 to 47d99ac Compare November 5, 2024 17:25
@danieldaquino
Copy link
Contributor

Thank you @scoder1747! I will take a look soon

@danieldaquino danieldaquino merged commit c4ee52f into damus-io:master Nov 11, 2024
@danieldaquino
Copy link
Contributor

Thank you @scoder1747! Changes look good, merged!

@alltheseas alltheseas linked an issue Nov 18, 2024 that may be closed by this pull request
2 tasks
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.

Bug: share image bugs tracker
2 participants