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
No internet connection banner does not disappear on regaining connection. #3485
Comments
Solution possibly related to #3281. |
I won't be able to resolve this issue because of some issues with the iOS simulator. iOS simulators do not propagate This comment is just as a PSA, ensure access to an iOS device before trying to solve this issue. |
Thanks @ishammahajan for filing this and digging into it! Since you weren't able to reproduce on Android (and I don't think I've seen this in my own regular use on my Android phone), it sounds like this is probably iOS-specific. Adding our label for that. |
It’s working correctly for me on iOS 13.3 with Zulip 26.20.143; the banner appears when I switch on airplane mode, and disappears when I switch it off. Has anyone noticed this still happening, and on which version? |
876f22d for issue #3602, released with 26.13.136 in November, 2019, was a revert of some code that animated the entrance/exit of the banner, but with a bug where it would get stuck and show up when it shouldn't. But symptoms were only reported on Android, and it seemed like it was possible to make the banner go away with further interactions. This may be no relation to the present issue — or, it was a bug whose root cause affected both platforms, and we didn't realize. We may be able to rule out one reason the root cause would be platform-specific. The animation was on the 'height' attribute, so the device-specific native driver was not being used; the animation had to have been implemented entirely in JavaScript, which runs on both platforms. (It's one of the caveats described in the Animated library's doc here), and you can check that Possibly, its symptoms were observed to be platform-specific because there was an Android-particular pattern triggering entrances and exits of the banner very rapidly; note, in the commit message,
I believe it was never concluded that #3602 had an Android-specific root cause. Could it be that some JavaScript run in that animation on both Android and iOS just can't handle very rapid state changes between "visible" and not? @ray-kraesig, any thoughts? |
I have no evidence against it! (As a hacky test, if we can manage to reproduce the bug on a physical Pie device, it might be possible to record the actual sequence of event times, and then issue mock disconnection events with similar timings in order to try to reconstruct the bug on an emulator, earlier-Android, or iOS device.) |
As discussed on chat, there have been multiple reports of the
No Internet Connection
banner not disappearing when it should, as the internet connection resets. To reproduce, login to the Zulip realm and turn off the internet connection. Wait for a few seconds and resume the connection. The banner should persist. (video)As pointed out helpfully by @borisyankov in the chat, these are a few pointers:
The text was updated successfully, but these errors were encountered: