Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#1734 closed enhancement (fixed)

[PATCH] Group Civs/Factions in civinfo page

Reported by: s0600204 Owned by: leper
Priority: Nice to Have Milestone: Alpha 12
Component: UI & Simulation Keywords:
Cc: Patch:

Description

As someone new to 0ad, I feel that that the list of Civilisations as viewable through the Civinfo UI (accessible from the main menu) can be a little confusing.

Why? Well, the website maintains that there are six unique Civs, but the list of civilisations in the civinfo ui runs to 13 entries. Although it is mentioned (in the forums) that some of these are 'Factions', the list in-game does not give any indication of which these are.

This patch alters the sorting of the list of civilisations in the Civinfo UI so they are grouped by their parent 'culture', with the Factions indented.

Attachments (1)

civinfo.patch (2.1 KB ) - added by s0600204 11 years ago.

Download all attachments as: .zip

Change History (4)

by s0600204, 11 years ago

Attachment: civinfo.patch added

comment:1 by Erik Johansson, 11 years ago

The web site is extremely outdated by now (and will be replaced soon, hopefully within the next couple of days as long as everything goes well), and the in-game number is probably more accurate, though it's not final either as some factions/civs might be added/removed before a final release.

Grouping the factions in the GUI is still probably a good idea, so I'll leave this ticket open and if that's all your patch does I'm sure someone will review it and add it soon :)

comment:2 by leper, 11 years ago

Owner: set to leper
Resolution: fixed
Status: newclosed

In 12905:

Group civs in info screen by culture. Fixes #1734.

comment:3 by leper, 11 years ago

Keywords: review removed

Thanks for the patch, but the code was rather strange as the only change needed was changing the sorting function a bit so I just went ahead and fixed this.

Note: See TracTickets for help on using tickets.