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

WIP: Refactor project into Swift Package #1

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

WIP: Refactor project into Swift Package #1

wants to merge 3 commits into from

Conversation

kevinrpb
Copy link

@kevinrpb kevinrpb commented Aug 2, 2019

  • Created a SPM package and set the ModalView part as a framework so that it can be imported from Xcode 11 as a dependency

  • Move the former project into another target as a demo

Let me know what you think and/or if I should change anything =D

@kevinrpb kevinrpb closed this Aug 2, 2019
@kevinrpb
Copy link
Author

kevinrpb commented Aug 2, 2019

Closed it because I noticed something else needing refactor in order to use as framework. Wille reopen when I finish it

Hey, @cyrilzakka, I got most of it working. However, I'm having some problems with the transformation applied when the modal is open.

It seems that when I try to change the modal position from outside, it won't detect the needed offset / opacity correctly. Do you have a moment to help me solve this? 😀

@kevinrpb kevinrpb changed the title Refactor project into Swift Package WIP: Refactor project into Swift Package Aug 2, 2019
@kevinrpb kevinrpb reopened this Aug 2, 2019
@cyrilzakka
Copy link
Owner

Will definitely take a look at this as soon as possible! Thank you for your work!

cyrilzakka added a commit that referenced this pull request Aug 5, 2019
- Fixes an issue with corner radius not updating properly.
- Uses ObservableObject in preparation for next update.
- Does not fix issues #1  and #2.
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.

2 participants