Opened 8 years ago

Closed 7 years ago

#3959 closed defect (fixed)

[PATCH] Missing icons for new buildings in alpha 21

Reported by: elexis Owned by:
Priority: Should Have Milestone: Alpha 21
Component: Art & Animation Keywords: patch simple
Cc: Patch:

Description

Some new buildings don't come with a new icon. This includes the briton crannog, royal stoa and possibly others. See also wiki:Alpha21

Attachments (2)

templeofvestaicon.patch (1.5 KB ) - added by Sandarac 8 years ago.
Adds Temple of Vista icon. Art by LionKanzen.
temple_vesta.png (39.4 KB ) - added by Sandarac 8 years ago.

Download all attachments as: .zip

Change History (21)

comment:1 by Lionkanzen, 8 years ago

I Need finish simple icons first, this need a lot a of time single by single.

by Sandarac, 8 years ago

Attachment: templeofvestaicon.patch added

Adds Temple of Vista icon. Art by LionKanzen.

by Sandarac, 8 years ago

Attachment: temple_vesta.png added

comment:2 by Sandarac, 8 years ago

Keywords: review patch added
Milestone: BacklogAlpha 21
Summary: Missing icons for new buildings Stoa and Crannog[PATCH] Missing icons for new buildings Stoa and Crannog

comment:3 by Lionkanzen, 8 years ago

I don't, nobodies are doing feedback or talking , may be I need a review first.

comment:4 by Niek, 8 years ago

Why did you choose a statue (of what I assume is a Vestal Virgin) to represent a temple building?

in reply to:  4 comment:5 by Lionkanzen, 8 years ago

Replying to niektb:

Why did you choose a statue (of what I assume is a Vestal Virgin) to represent a temple building?

I suposse most closely to Vesta, the more easy/ cheap to do is take the building and create an icon...

comment:6 by Lionkanzen, 8 years ago

Or try with symbols and coins representation but with same as civic center style artwork http://www.romanreligion.org/content/wp-content/uploads/VestaTemple_coinreverse.jpg

comment:7 by Niek, 8 years ago

Keywords: review removed

I'm removing it from the review queue since it's not clear whether the current for the Vesta Temple is a good representation (and since that's the only icon ATM)

comment:8 by Lionkanzen, 8 years ago

May be we need, wait for another artist. :)

comment:9 by Lionkanzen, 8 years ago

Cc: Lionkanzen removed

comment:10 by elexis, 8 years ago

Keywords: review added

niektb committed the temple of vesta icon in r18593.

Someone should commit one of the two crannog icons this release:

https://wildfiregames.com/forum/index.php?/topic/20811-buildings-portrait-process-and-little-theory/&page=3

comment:11 by fatherbushido, 8 years ago

The persian hall and perhaps also that persian gates are also missing.

comment:12 by elexis, 8 years ago

Crannog icon committed in r18732

comment:13 by elexis, 8 years ago

Keywords: simple added; review removed
Milestone: Alpha 21Backlog

comment:15 by elexis, 8 years ago

Keywords: review added
Milestone: BacklogAlpha 21

New icons available in that thread. The persian hall one seems to be ready to commit.

comment:16 by elexis, 7 years ago

Keywords: review removed

Persian hall icon committed in r18795

comment:17 by Niek, 7 years ago

In 18855:

New icon for the Persian Ishtar Gate, made by Lion.Kanzen. Refs #3959

comment:18 by elexis, 7 years ago

In 18893:

Sentry tower icon by Lion.Kanzen, refs #3959.

comment:19 by elexis, 7 years ago

Resolution: fixed
Status: newclosed
Summary: [PATCH] Missing icons for new buildings Stoa and Crannog[PATCH] Missing icons for new buildings in alpha 21
Note: See TracTickets for help on using tickets.