Project

General

Profile

Bug #693242

Several tilesets still reference "airfield" in extrastyles (should be "airstrip")

Added by Griffon Spade about 1 month ago. Updated 29 days ago.

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

0%


Description

Erroneously references base.airfield sprite in hexemplio.tilespec. Elsewhere is base.airstrip. (likely pre-standardization carry-over)

Line 282:
"base.airfield", "3Layer"

Should be:
"base.airstrip", "3Layer"

m-30-tileset-airfield-refs.patch View (2.16 KB) Jacob Nevins, 2017-08-26 06:34 PM

26-tileset-airfield-refs.patch View (2.17 KB) Jacob Nevins, 2017-08-26 06:34 PM

History

#1 Updated by Jacob Nevins about 1 month ago

  • Subject changed from Hexemplio.tilespec 2.6+ error to Several tilesets still reference "airfield" in extrastyles (should be "airstrip")
  • Category set to Art
  • Status changed from New to In Progress
  • Assignee set to Jacob Nevins
  • Target version set to 2.6.0-beta1

Also Cimpletoon (default tileset), Toonhex, Alio.

I think we've been getting away with "airstrip" effectively not defining an extrastyle because tileset_setup_extra() silently falls back to the extrastyle for the alternate graphics (which for civ2civ3 is "airbase"), which in these tilesets happens to have the same extrastyle ("3Layer"), while still using "airstrip" sprites; that's probably a bit lax of tileset_setup_extra().

#2 Updated by Jacob Nevins about 1 month ago

  • Tracker changed from Task to Bug

#3 Updated by Jacob Nevins about 1 month ago

#4 Updated by Jacob Nevins 29 days ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF