Opened 2 years ago

Closed 23 months ago

#6503 closed enhancement (fixed)

Han Chinese siege towers issues

Reported by: Langbart Owned by: Stan
Priority: Nice to Have Milestone: Alpha 26
Component: Art & Animation Keywords:
Cc: Patch:

Description (last modified by marder)

Two issues

  • Attack

IRC 0ad-dev ​29/Apr/22

18:26 < phosit> It looks weird if the siege towers of Han Chinese attack: The lances stab even if the tower is far away from the enemy
18:28 < phosit> looks like attack_ranged animation is just a placeholder?

See GIF below, the tower is manned by women and does not shoot any arrows, the moving spears give the false impression that it is some kind of deadly weapon even though it is completely harmless.

  • Garrison flag

It is a bit short. Flag hides when the tower is idle, see GIF below.

Attachments (7)

towergif.gif​ (900.8 KB ) - added by Langbart 2 years ago.
short_flag.jpg​ (79.0 KB ) - added by Langbart 2 years ago.
flag.gif​ (1.3 MB ) - added by Langbart 2 years ago.
half.jpg​ (340.3 KB ) - added by Langbart 2 years ago.
flagmyface.jpg​ (339.7 KB ) - added by Langbart 2 years ago.
han_s_tower_attachpoint_flag.jpg​ (326.0 KB ) - added by Langbart 2 years ago.
han_siege_tower.dae​ (594.3 KB ) - added by Langbart 2 years ago.
han_siege_tower.dae

Change History (18)

by Langbart, 2 years ago

Attachment: towergif.gif​ added

by Langbart, 2 years ago

Attachment: short_flag.jpg​ added

comment:1 by Stan, 2 years ago

Was originally supposed to be a melee attack...

comment:2 by wowgetoffyourcellphone, 2 years ago

I honestly wish we could give Siege Towers a capture attack along with their ranged attack.

comment:3 by Freagarach, 2 years ago

​Phab:D1819 ;)

comment:4 by Langbart, 2 years ago

Description: modified (diff)

adding a GIF

by Langbart, 2 years ago

Attachment: flag.gif​ added

comment:5 by Freagarach, 2 years ago

Owner: set to marder
Patch: → Phab:D4655

comment:6 by marder, 2 years ago

Resolution: → fixed
Status: new → closed

In 26897:

Fix issues with han siege tower

reported by @Langbart ; Fixes #6503 , also adjusts the footprint to fit better to the actor

comments by: @Stan
accepted by : @phosit

Differential revision: ​https://code.wildfiregames.com/D4655

in reply to:  description comment:7 by Langbart, 2 years ago

Resolution: fixed
Status: closed → reopened

Replying to Langbart:

  • Garrison flag

It is a bit short.

The prop point (prop_garrisoned) in the han_siege_tower.dae needs to be moved a bit higher.

My file did work but it was only 50% as long as the original.

Related forum threads:

Last edited 2 years ago by Langbart (previous) (diff)

by Langbart, 2 years ago

Attachment: half.jpg​ added

by Langbart, 2 years ago

Attachment: flagmyface.jpg​ added

by Langbart, 2 years ago

Attachment: han_s_tower_attachpoint_flag.jpg​ added

by Langbart, 2 years ago

Attachment: han_siege_tower.dae​ added

han_siege_tower.dae

comment:8 by Freagarach, 2 years ago

Resolution: → fixed
Status: reopened → closed

Fixed in [26897].

comment:9 by Freagarach, 2 years ago

Resolution: fixed
Status: closed → reopened

Sorry for not reading.

comment:10 by marder, 2 years ago

Component: UI – In-game → Art & Animation
Description: modified (diff)
Owner: marder removed
Patch: Phab:D4655
Status: reopened → new

comment:11 by Stan, 23 months ago

Owner: set to Stan
Resolution: → fixed
Status: new → closed

In 26910:

Slightly improve the look of the chinese siege tower. Raise the garrison flag. Remove useless unused materials from the daes.
Fixes #6503
Reported by: @Langbart

Note: See TracTickets for help on using tickets.