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

Animating Protocol is Package-Protected #112

Closed
cody1024d opened this issue Sep 29, 2016 · 9 comments
Closed

Animating Protocol is Package-Protected #112

cody1024d opened this issue Sep 29, 2016 · 9 comments

Comments

@cody1024d
Copy link
Contributor

cody1024d commented Sep 29, 2016

I was trying to implement a custom view (very similar to the one used by default, but at a bigger size, and some margins on the title label), however, the animation protocol is currently not public, and so I cannot implement it with my custom view. Would you be open to a pull request to fix this?

It seems like, without the above, custom views would not work nearly as well.

@objc protocol PKHUDAnimating {

    func startAnimation()
    optional func stopAnimation()
}
@cody1024d
Copy link
Contributor Author

On that note, the actual animation static class also does not expose the discrete rotation animation as public, either. This would be nice for custom views which are only different in small ways when compared to the provided progress view.

public final class PKHUDAnimation {

    static let discreteRotation: CAAnimation = {
...
...

@PGLongo
Copy link
Collaborator

PGLongo commented Dec 20, 2016

Hi @cody1024d sorry for the delay. Are you still interested in this feature?

@cody1024d
Copy link
Contributor Author

I definitely would. I am still open to making a PR for it too, it would be relatively quick! (Considering I just have to change the scoping on a few of these properties). I would have time to do it in the coming days, if that's something you'd be ok with?

@PGLongo
Copy link
Collaborator

PGLongo commented Dec 21, 2016

For me is ok. But consider to make differents PR to keep everything simple. First of take care of this issue and the problem of the protocol, after that we could discuss on how to proceed to allow changing the size and other stuff of the HUD.

@cody1024d
Copy link
Contributor Author

Good idea. I will make separate PRs, so each change can be discussed on it's own merit. I should have some time today to do them, but no promises! :)

@PGLongo
Copy link
Collaborator

PGLongo commented Dec 21, 2016

Later I will create an issue where we can discuss about the resize.

@cody1024d
Copy link
Contributor Author

@PGLongo Just made a PR with the protocol changes :)

@PGLongo
Copy link
Collaborator

PGLongo commented Dec 21, 2016

Merged

@PGLongo PGLongo closed this as completed Dec 21, 2016
@tommylo
Copy link

tommylo commented Mar 3, 2017

Still can not import PKHUDAnimating protocol

Use of undeclared type "PKHUDAnimating"

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

No branches or pull requests

3 participants