[http://sourceforge.net/p/ufoai/bugs/2998 Item 2998] imported from sourceforge.net tracker on 2013-01-28 19:54:51

The current team selection interface is very confusing and should be removed (after branching) or improved for 2.4.

The main problem is that there is no clear text to indicate which team is currently selected. In most cases it seems to work as one might guess, but a lot of guessing is involved to figure out what to do. After loading a team and editing it, it's not clear that the loaded team is now selected. If the user loads a team and edits it, then selects a default team from the team selectbox, the team that was loaded and edited will still be used. It's not clear that the user must click on the NEW button after selecting a default team from the selectbox. This is especially confusing because if no saved team has been loaded, the user can simply select the desired default team and click START and he will begin the game with his selected team. He must only click NEW if he has already loaded a saved team.

I'd be willing to take a crack at improving it if the *cvar:cl_teamdef was set up to always contain the name of the currently selected team (currently it is only updated when the selectbox is changed). This would have to update on the following events:

- When the player loads a saved team
- When the player clicks NEW to generate a new team
===== Comments Ported from Sourceforge =====

====== tlh2000 (2011-09-06 13:10:22) ======

the multiplayer part is the same and should be fixed/improved, too
====== tlh2000 (2011-09-07 05:30:37) ======

You can what to be the value of cl_teamdef? We maybe (depending on your wish of the value) can't use cl_teamdef here (unless i got you wrong), as this is only for holding values like "civilian", "alien" and "phalanx". not the current selected team (team0.mpt or whatever)
====== tlh2000 (2011-09-07 05:32:55) ======

sorry: "You want what to be the value... "
====== tlh2000 (2011-09-07 05:33:56) ======

there is also mn_teamname as the current loaded team name - this is set in GAME_LoadTeam and should be used for skirmish and multiplayer
====== tlh2000 (2011-09-07 05:34:17) ======

... but this is not (yet) set for saving a team
====== tlh2000 (2011-09-07 05:37:14) ======

mn_teamname is now also set on saving: 28ebc1bad48d158ccfbd2d330cae8c17fe059e9e
====== tlh2000 (2011-09-07 05:41:52) ======

i'm not sure whether you can use this cvar - the names must not be unique. so comparing mn_teamname with mn_slotXX is not enough imo. maybe we need a confunc to mark the loaded slot. (if i got your initial but report right ;) )