Fix BTC status incorrectly hidden on splash screen #2567
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue: When launching a new Bisq instance (i.e. new data directory) and
![image](https://user-images.githubusercontent.com/603793/54773294-83e5e400-4bc6-11e9-8f0c-1666fc8d1e2f.png)
remaining on the user agreement screen for >90 seconds while reading it,
once you accept the agreement the BTC status was not being shown
on the splash screen.
Cause: showTorNetworkSettingsTimer gets triggered after 90 seconds
and since Tor is not started until after accepting the user agreement,
it was incorrectly assuming that Tor is not working and as a result
hiding the BTC status.
Fix: Don't hide the BTC status in showTorNetworkSettingsTimer.
![image](https://user-images.githubusercontent.com/603793/54773299-89432e80-4bc6-11e9-93ee-c3d1d4da1498.png)
If there is an issue with Tor, splashP2PNetworkErrorMsgListener handles
![image](https://user-images.githubusercontent.com/603793/54773343-9bbd6800-4bc6-11e9-9327-4ede800751d4.png)
hiding the BTC status.