Connection Dialog Should Display Ping Issues (timeouts, packetloss)

[Kissaki]

I had internet connection issues the last two days and noticed the server stats are only updated on successful pings. That means that if you had 1 successful 30ms ping with user count and then none for 10 minutes the displayed info is 10 minutes old and wrong (/does not indicate connection issues).

If we start with timeouts we do not display anything. Maybe some negative feedback would be useful then as well (use case: user has only 1 favourite server; does not see that we ping unsuccessfully, connect button is greyed out).

[Kissaki]

/spend 35min

Ping logic is so complex code /o\

[topas-rec]

I'm still doing old school network gaming sessions and we're happy to be able to use mumble for voice chatting there.

Everytime a new user uses mumble I have to explain that the connect button will change to active after some time and that they have to be patient.

If this issue is about showing some more information in the GUI while the "server testing" is happening - I am supporting it.

Additionally showing "how the server test currently goes" (I think that's what was suggested above) is also fine, but an add-on from my perspective.

~~By the way, where is the "greyed out button" and the "server testing" done in the code? Can someone point me there (faster than I could)?~~

[davidebeatrici]

I believe the button should always be active. We can show the user a message box explaining the error if connection to the server is not possible or fails.

[topas-rec]

That would less unexpected and therefore I prefer this way, too.