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

Adding support for macOS by using shims #137

Closed
wants to merge 6 commits into from
Closed

Adding support for macOS by using shims #137

wants to merge 6 commits into from

Conversation

Fab1n
Copy link

@Fab1n Fab1n commented Feb 14, 2017

Adds complete support for using PKHUD under macOS.
Changes are almost all additive and held simple by adding a shim file to bring both platform APIs together.

Use the demo app to check if all huds work smoothly and do what they say.
Then please check logic and code style and see if it all fits.

Fabian Renner added 4 commits February 8, 2017 16:07
This allows PKHUD to be used across iOS and macOS

Note: WindowRootViewController has been dropped, as it is not used anymore.
On macOS showing a hud in a window sheet lead to black corners where the VisualEffectView should have transparent rounded corners
@PGLongo
Copy link
Collaborator

PGLongo commented Feb 14, 2017

Thanks for this HUGE PR. I will give a try ASAP.

@PGLongo PGLongo self-assigned this Feb 14, 2017
@Fab1n
Copy link
Author

Fab1n commented Feb 22, 2017

some progress there?

@PGLongo
Copy link
Collaborator

PGLongo commented Feb 22, 2017

Not so many. I thinks it would be better to change the branch to feature/macOS and work there for the next release.

@Fab1n
Copy link
Author

Fab1n commented Feb 24, 2017

closing, due to working in a feature branch where this (#141) PR tries to add the changes

@Fab1n Fab1n closed this Feb 24, 2017
@PGLongo
Copy link
Collaborator

PGLongo commented Feb 24, 2017

Thanks a lot

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