#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)
Change History (4)
by , 11 years ago
Attachment: | civinfo.patch added |
---|
comment:1 by , 11 years ago
comment:3 by , 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.
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 :)