Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#1333 closed enhancement (fixed)

[PATCH] Add map name and size to game summary screen

Reported by: rdxi Owned by: leper
Priority: Nice to Have Milestone: Alpha 11
Component: UI & Simulation Keywords: simple gui
Cc: Patch:

Description


Attachments (2)

summary_map_name.patch (2.3 KB ) - added by Alex 12 years ago.
A first attempt at implementing this. The map size is displayed as a number.
summary_map_name_v2.patch (3.2 KB ) - added by Alex 12 years ago.
Updated patch to display the map size as a string.

Download all attachments as: .zip

Change History (8)

comment:1 by historic_bruno, 12 years ago

Keywords: simple gui added

comment:2 by Alex, 12 years ago

Hello,

I've written a small patch to implement this. I've also taken the liberty of slightly adjusting the GUI layout of the summary screen ( just the top panel ). I'm also not aware of how to interpret the map size. The result I get from

Engine.GetMapSettings().Size

is an integer, but I suppose that has to be interpreted as a string ( Tiny, Normal, etc. ). If anyone has any ideas, I'm looking forward to them.

by Alex, 12 years ago

Attachment: summary_map_name.patch added

A first attempt at implementing this. The map size is displayed as a number.

by Alex, 12 years ago

Attachment: summary_map_name_v2.patch added

Updated patch to display the map size as a string.

comment:3 by Alex, 12 years ago

Ok, I've managed to display the size of the map a string, taken from "data/mods/public/simulation/data/map_sizes.json". If the map is a scenarion, the string "Scenario" is used instead of the map size.

Last edited 12 years ago by Alex (previous) (diff)

comment:4 by Alex, 12 years ago

Keywords: review added
Milestone: BacklogAlpha 11
Summary: Add map name and size to game summary screen[PATCH] Add map name and size to game summary screen

comment:5 by leper, 12 years ago

Owner: set to leper
Resolution: fixed
Status: newclosed

In 12020:

Display map name and size on summary screen. Based on patch by hapsky. Fixes #1333.
Play victory and defeat music.
Replace celts with gauls.

comment:6 by leper, 12 years ago

Keywords: review removed

Thanks for your patch.

Note: See TracTickets for help on using tickets.