Constructed Game launch screen / Deck chooser revamp

[spr]

The aim is to implement this -

but I will try to commit in stages since there is quite a lot of changes.

Cheers,
Steve

[spr]

r23358
- Added game type combo to replace the ambiguous "Change Player Type" buttons.
- Remembers game type between sessions.
- Moved "Random" button to bottom of decks list so screen will resize gracefully with no components going AWOL;
- Declutter screen by moving game options to "Game" menu.

Cheers,
Steve

[Max mtg]

will it support folders for constructed deck selection?

[spr]

will it support folders for constructed deck selection?

Yes - it uses the same code for discovering decks as used now.

Cheers,
Steve

[spr]

r23361

• Replaced deck category radio buttons with combo. As well providing more space for the decks list this should make it easier to add new categories in future without requiring any change to the UI.
• Deck list transparency.

Cheers,
Steve

[spr]

Suggestion : the default setting for the Constructed launch screen should be a Human v Computer game with the Human deck category set to Preconstructed and the Computer set to Random Color.

My understanding is that the Preconstructed decks are aimed at new players so it would seem a more natural default setting. Anyone have objections if I update it to these defaults?

Cheers,
Steve

[Max mtg]

That label 'human v computer' is way too large, moreover it's repeating the information we can read below in deck selection headers

[spr]

r23370 : Feedback update
- Restored buttons to combos.
- Made the game type combo slightly smaller.
- Removed deck headings.

Cheers,
Steve

[spr]

r23370 : Saved state update
- Constructed game now remembers the last deck played for both left and right decks. This state will be restored each time Forge starts. It does not distinguish between game type.
- Default human deck type is Preconstructed decks.

Mulitple instances of FDeckChooser are used in a number of places. Currently its state (that is the deck category and selected deck(s)) can be saved by calling SaveState(), but for that to work you need to pass in an FPref setting via the existing initialize() method. The Constructed screen calls SaveState() when the user clicks Start. Currently, only constructred screen makes use of this feature.

Cheers,
Steve