Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

new (1.17) animation for spearman/swordsman #686

Open
white-haired-uncle opened this issue Mar 6, 2024 · 1 comment
Open

new (1.17) animation for spearman/swordsman #686

white-haired-uncle opened this issue Mar 6, 2024 · 1 comment

Comments

@white-haired-uncle
Copy link
Collaborator

white-haired-uncle commented Mar 6, 2024

Got an image error due to spearman-attack-ranged changes. I don't understand animations and don't feel like trying to figure this out.

1.16/data/core/images/units/human-loyalists/spearman-attack-ranged.png
1.16/data/core/images/units/human-loyalists/spearman-attack-ranged2.png
1.17/data/core/images/units/human-loyalists/spearman-attack-ranged1.png
1.17/data/core/images/units/human-loyalists/spearman-attack-ranged2.png
1.17/data/core/images/units/human-loyalists/spearman-attack-ranged3.png

I'm guessing the idea is not just to replace the missing one, but to update the sequence?

Effected: units/demon_infiltrators*.cfg

See also units/human-loyalists/spearman-attack-s-*

EDIT: Found more. I'm submitting them to this thread: https://forums.wesnoth.org/viewtopic.php?t=57288

@Dugy
Copy link
Owner

Dugy commented Mar 6, 2024

Copypasting the animation from the core units over the old one is probably the best approach here.

I am sorry for not doing it myself, but the unstable versions have become rather inconvenient to set up on Linux once they introduced flatpak, so I would need to spend quite a lot of effort to test it.

@white-haired-uncle white-haired-uncle changed the title new (1.17) animation for spearman new (1.17) animation for spearman/swordsman Mar 8, 2024
white-haired-uncle added a commit that referenced this issue May 10, 2024
Fixed spearman/swordsman and provides an example for fixing others

Ref #686
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants