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

Set Window isHidden to True on Init #165

Merged
merged 3 commits into from
May 9, 2017

Conversation

Veronique
Copy link
Contributor

@Veronique Veronique commented May 9, 2017

When HUD first get initialized, it is not hidden. As a result, checking HUD.isVisible returns true even though it has never been presented. Setting isHidden to true on Window.swift resolves this.

The particular case I am addressing is for deeplinking into an app. I only want to proceed with the deep link if there is not a non-interruptible operation in progress (easily identified by a visible HUD).

@Veronique
Copy link
Contributor Author

Veronique commented May 9, 2017

Note that the tests pass when testing locally. I am changing the time interval on the tests to address the CI time-out failure.
screen shot 2017-05-09 at 11 41 03 am

@PGLongo PGLongo self-requested a review May 9, 2017 16:08
Copy link
Collaborator

@PGLongo PGLongo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The timeout of 3 seconds won't work in local (in my case) I got a failure in testTextHUD.

Please increase to 3.5 or remove the commit and I will fix with another push

@PGLongo PGLongo merged commit 95a82d3 into pkluz:master May 9, 2017
@PGLongo
Copy link
Collaborator

PGLongo commented May 10, 2017

Unfortunately there are still problems with Travis. I should investigate

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