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
cgame: preselect a map in server creation window, remember it #2883
base: master
Are you sure you want to change the base?
Conversation
f88a7f1
to
5bf062a
Compare
…ation window Also unify the look of the callvote map window with the server_create one.
0a5c009
to
c4d4050
Compare
I added a commit that forces the height of the levelshot. I forced This prevents the server creation window size to change dynamically while scrolling maps, hence preventing the “Start” button to move up and down while scrolling map, and it would prevent a nasty map with a nasty |
c6f01c9
to
b0f7606
Compare
I don't have time to review or test, but I would ask to ensure that this does not cause a map pak to be loaded before opening the map menu. |
This shouldn't do that |
@@ -34,6 +34,8 @@ Maryland 20850 USA. | |||
|
|||
#include "cg_local.h" | |||
|
|||
static Cvar::Cvar<std::string> g_defaultMap("g_defaultMap", "default map to play", Cvar::ARCHIVE, "plat23"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be cg_
{ | ||
if ( rocketInfo.data.mapList.size() != 0 ) | ||
{ | ||
CG_Rocket_BuildMapList( nullptr ); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Build the list only if it is already nonempty? Seems weird. Probably this shouldn't be here at all?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah no it should be == 0
, well, I copypasted CG_Rocket_GetResolutionListIndex
the cargo cult way, so I don't know if that code is needed, but when duplicating, that should be ==
. I'll fix that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since the code worked even with !=0
this shouldn't be needed at all.
Preselect a map in server creation window, this means a newcomer has less actions to do to create a server when trying the game for the first time, and then remember the last started map.