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

Fix behavior inconsistencies #576

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

Conversation

morozkin
Copy link

Hi!

With this PR I fixed several behavior bugs and added a test for these cases.

They are:

  1. If you try to hide a hud without animation, the alpha value for the backgroundView will be equal to 1.0 while it must be equal to 0.0
  2. If you have a hud with an animation type set to MBProgressHUDAnimationZoomIn and you hide this hud animatedly, trying to show it non-animated will end up in having a transform applied to the hub as a result of animated hide which is undesirable behavior.

@morozkin
Copy link
Author

@matej Can you have a look at this PR?

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.

1 participant