Project

General

Profile

Feature #657385

Outpost and Airstrip activity icons

Added by Marko Lindqvist 7 months ago. Updated 3 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Art
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:

History

#1 Updated by Jacob Nevins 5 months ago

Words from gna patch #5647:

cazfi: Other base types use small version of the base as the activity icon, so should [Outpost] and Airstrip.

jtn: We sort of decided that certain tilesets don't get per-extra activity icons as part of their style (amplio, hex2t, isophex). If we stick with that, they don't need fixing for this either (of course they need to display the appropriate letters).

#2 Updated by Jacob Nevins 3 months ago

Confirmed that all tilesets which already had non-letter fortress/airbase activity icons also have distinct outpost/airstrip activity icons.

For the remaining tilesets, the attached patch attaches explicit unit.outpost/airstrip definitions to the existing "F"/"A" activity icons. These were already in use for civ2civ3 through fallbacks, but giving them explicit definitions makes it possible for some other ruleset to use outpost/airstrip as fallbacks for something else, as well as being generally less confusing (to me at least).

#3 Updated by Marko Lindqvist 3 months ago

Does that "alias" format really work? (I know some tileset have had the construct). It seems to contradict general secfile format, as the table header lists only "y", "x", and "tag" columns

#4 Updated by Jacob Nevins 3 months ago

Experimentally, it appears to.

In scan_specfile(), "tag" is looked up with secfile_lookup_str_vec(). There's some words about this in the comments in registry_ini.c about 'tabular format'.

#5 Updated by Marko Lindqvist 3 months ago

Jacob Nevins wrote:

There's some words about this in the comments in registry_ini.c about 'tabular format'.

It's documented in README.graphics, once you know to look for it (format example could show this kind of case too)

#6 Updated by Jacob Nevins 3 months ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF