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

Include iPadCoordinates in options for share() and shareWithOptions() #1057

Merged
merged 2 commits into from
Sep 17, 2019

Conversation

TaDaa
Copy link
Contributor

@TaDaa TaDaa commented Sep 17, 2019

Allows coordinates to be driven through config rather than separate function and puts popover sourceRect back in a working state for iPad.

@EddyVerbruggen EddyVerbruggen added this to the 5.6.1 milestone Sep 17, 2019
@EddyVerbruggen EddyVerbruggen merged commit 639b1e2 into EddyVerbruggen:master Sep 17, 2019
@EddyVerbruggen
Copy link
Owner

Ha, cool! Thank you :)

@EddyVerbruggen
Copy link
Owner

Hi @TaDaa can you please check #1066 - it may have been caused by this change.

@TaDaa
Copy link
Contributor Author

TaDaa commented Oct 16, 2019

Think you are right -- should be an easy fix, will post there and get you a pr on it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants