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)
Change History (21)
comment:1 by , 8 years ago
by , 8 years ago
Attachment: | templeofvestaicon.patch added |
---|
Adds Temple of Vista icon. Art by LionKanzen.
by , 8 years ago
Attachment: | temple_vesta.png added |
---|
comment:2 by , 8 years ago
Keywords: | review patch added |
---|---|
Milestone: | Backlog → Alpha 21 |
Summary: | Missing icons for new buildings Stoa and Crannog → [PATCH] Missing icons for new buildings Stoa and Crannog |
comment:3 by , 8 years ago
I don't, nobodies are doing feedback or talking , may be I need a review first.
follow-up: 5 comment:4 by , 8 years ago
Why did you choose a statue (of what I assume is a Vestal Virgin) to represent a temple building?
comment:5 by , 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 , 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 , 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:9 by , 8 years ago
Cc: | removed |
---|
comment:10 by , 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:
comment:13 by , 8 years ago
Keywords: | simple added; review removed |
---|---|
Milestone: | Alpha 21 → Backlog |
comment:14 by , 8 years ago
comment:15 by , 8 years ago
Keywords: | review added |
---|---|
Milestone: | Backlog → Alpha 21 |
New icons available in that thread. The persian hall one seems to be ready to commit.
comment:19 by , 7 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Summary: | [PATCH] Missing icons for new buildings Stoa and Crannog → [PATCH] Missing icons for new buildings in alpha 21 |
I Need finish simple icons first, this need a lot a of time single by single.