Opened 12 years ago

Closed 8 years ago

Last modified 8 years ago

#1090 closed enhancement (fixed)

[PATCH] Improve load game UI

Reported by: historic_bruno Owned by: mimo
Priority: Should Have Milestone: Alpha 21
Component: UI & Simulation Keywords:
Cc: Patch:

Description (last modified by historic_bruno)

Currently when loading a saved game, it shows a list with timestamp and map name for each game. I would like to see at minimum:

  • Timestamp
  • Map name
  • User's description of the game
  • Players
    • Name / player number
    • Civ
    • Status (defeated/active)
  • Elapsed time

Possible enhancements:

  • Small screenshot
  • Game version; mod(s) used

It doesn't all have to be shown on one line, but rather have a common UI panel to display details for the currently selected game.

Attachments (8)

loadsave.patch (3.1 KB ) - added by mimo 8 years ago.
with fixes following IRC comments
loadsave-v2.patch (6.4 KB ) - added by mimo 8 years ago.
new version with map preview
load.png (841.8 KB ) - added by mimo 8 years ago.
screenshot of the new panel
loadsave-v3.patch (16.9 KB ) - added by mimo 8 years ago.
loadsave-v4.patch (18.5 KB ) - added by mimo 8 years ago.
playercomponent_savegamesummaryreplaycleanup.patch (45.1 KB ) - added by elexis 8 years ago.
cleans up related and related related code in several ways, most notably changes the player component to trigger the defeat directly instead of sending a message to it to remove duplicate messages.
playercomponent_savegamesummaryreplaycleanup_v2.patch (54.6 KB ) - added by elexis 8 years ago.
fixed order of execution bug, added more cleanup because the patch wasnt big enough
playercomponent_savegamesummaryreplaycleanup_v3.patch (25.7 KB ) - added by elexis 8 years ago.
split unrelated parts

Download all attachments as: .zip

Change History (21)

comment:1 by Philip Taylor, 12 years ago

Also it should be possible to delete saved games from the UI, since people apparently like keeping their list clean.

comment:2 by ben, 12 years ago

In 11233:

Adds saved game deletion. Refs #1090
Documents a few things related to saved games.

comment:3 by historic_bruno, 11 years ago

Description: modified (diff)

#1683 was a duplicate.

comment:4 by historic_bruno, 11 years ago

Description: modified (diff)

r13579 added saved game descriptions

comment:5 by historic_bruno, 11 years ago

Description: modified (diff)

comment:6 by mimo, 8 years ago

Component: Core engineUI & Simulation
Keywords: review added; gui removed
Summary: Improve load game UI[PATCH] Improve load game UI

by mimo, 8 years ago

Attachment: loadsave.patch added

with fixes following IRC comments

comment:7 by elexis, 8 years ago

  • translate("Players: %(players)")
  • The tooltip doesn't state anything that isn't self-evident
  • A map preview would be great
  • Didn't test, but I doubt merging all these infos into a single string without newlines will show up correctly, especially with 8 players. Shouldn't there be labels like in the gamesetup/lobby/replay menu?

comment:8 by elexis, 8 years ago

Keywords: review removed

by mimo, 8 years ago

Attachment: loadsave-v2.patch added

new version with map preview

by mimo, 8 years ago

Attachment: load.png added

screenshot of the new panel

by mimo, 8 years ago

Attachment: loadsave-v3.patch added

comment:9 by mimo, 8 years ago

Keywords: review added
Milestone: BacklogAlpha 21

added some info on players (taken from the replay menu, with some common parts moved to gui/common)

by mimo, 8 years ago

Attachment: loadsave-v4.patch added

comment:10 by mimo, 8 years ago

Owner: set to mimo
Resolution: fixed
Status: newclosed

In 18409:

improve load game UI, fixes #1090

comment:11 by mimo, 8 years ago

Keywords: review removed

comment:12 by elexis, 8 years ago

In 18422:

Savegame GUI cleanup..

Move duplicate reallyDeleteGame to shared file.
Inline getPlayerInfoText since it is used only once, refs #1090.
Merge deleteGame and deleteGameWithoutConfirmation.
Use let instead of var.

by elexis, 8 years ago

cleans up related and related related code in several ways, most notably changes the player component to trigger the defeat directly instead of sending a message to it to remove duplicate messages.

by elexis, 8 years ago

fixed order of execution bug, added more cleanup because the patch wasnt big enough

by elexis, 8 years ago

split unrelated parts

comment:13 by elexis, 8 years ago

In 18440:

Savegame / summary screen / replay menu cleanup, refs #1090.

Savegame / replay menu selected game information:

Show players who won the game (not only defeat).
Show translated civs instead of the civ code.
Use a translation for "%(team)s:\n%(playerDescriptions)s".
Merge nearly duplicate objects passed to formatPlayerInfo.
Merge deleteGame and deleteGameWithoutConfirmation.

Summary screen:

Construct the summary screen title in place, not in the session, nor replay menu.
Remove g_Players from the replay metadata since all relevant information is contained in the sim data.
Split g_GameData into gui and sim.
Access g_GameData instead of data for consistency.
Move code from XML to JS.
Don't use victory music when observers enter the summary screen (just keep the current track like when exiting the game as a player).
Rename updateObjectPlayerPosition to initPlayerBoxPositions.

Note: See TracTickets for help on using tickets.