Server browser annoyance

[y00tz]

Whenever you have your list sorted by ping, anytime you select a server, I think the browser refreshes?  So if you ever click a server, the list scrambles and the server you clicked on jumps around too.

It took me a while to realize I was sorted by ping the first time this happened...

[KnacK]

* KnacK thinks y00tz needs to quit drinking when he plays DP....

:p

[Eiii]

When you click on a server, it has to send the status request and get all the players and scores and stuff back. This also updates the ping, which always is more than in used to because the status request is bigger than the info request that the server browser normally sends. It might be nice to have the browser not update ping when you click on it, though.

[y00tz]

Haha KnacK if only... if only...

I agree Eiii. (Wow that's gotta be a first)  At the very least, it shouldn't refresh when in 'sort ping' mode.

[jitspoe]

So fix it.

Or vote for it, I guess.

[y00tz]

So fix it.

What if we did this:

Current code, Line 952

Code: [Select]

SortServerList();
If that line is commented out, when it's in 'sort by ping' mode, it will update the ping of the server you're viewing as you click.  However, it won't update ALL the pings, and nor does it sort, therefore it won't move all the servers around.

Proposed code, line 952

Code: [Select]

// SortServerList();
-y00tz

edit: serverbrowserGUI.cpp... in case you didn't know..

[jitspoe]

Won't that stop sorting when you refresh the list, too?