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

Revisions to bobbing animations #5388

Merged
merged 11 commits into from
Dec 28, 2020
Merged

Conversation

doofus-01
Copy link
Member

I've never been happy with the heaving motions in the standing/bobbing animations of some of the sprites, for example the spearman and orc archer. However, the heavy breathing might make sense if the unit was physically failing, so in this PR I've got the filter to be < 1/2 HP-max (just to pick a number).

In principle, a similar distinction/filter could be made for healthy units, like the saurians dancing (just as an example of an existing standing animation that isn't heaving).

Filtering standing animations like this would multiply the needed animations by 2-3X, but we were never going to get all standing sprites animated anyway, and this would make future contributions more conceptually clear (dancing-->subtle shifting-->heaving).

In this PR, I've initially added the Orc Grunt and Archer. Before committing, I'd move other existing animations similarly, and add some new standing animations. But only if the concept makes sense, or if it would help the concept make sense.

@doofus-01 doofus-01 added Graphics Issues that involve the graphics engine or assets. Units Issues that involve unit definitions or their implementation in the engine. labels Dec 26, 2020
@Pentarctagon
Copy link
Member

Would this affect idle animations at all?

@doofus-01
Copy link
Member Author

It would prevent idle animations from firing if the unit is "wounded", but otherwise it wouldn't.

At this point in the PR, a spearman with >1/2 HP just stands there, and occasionally does the idle animation. A spearman with <= 1/2 HP starts heaving and never does the idle animation.

1/2 HP is probably too high, but good for testing.

@doofus-01
Copy link
Member Author

doofus-01 commented Dec 28, 2020

Now the orcish grunt is how I see this going, at least for heaving animations - there's still room for non-wounded standing animation.

If there are no objections, I will finish reviewing which existing bobbing animations are wounded heaving vs which are something else (like the ghosts or saurians aren't heaving), then run wmlindent and merge this.

EDIT:Well, also lower wounded threshold to ~1/3 HP-max.

@Pentarctagon
Copy link
Member

Makes sense to me. I'd also say if it comes up that the regular idle animations are a higher priority since they'll be the more commonly seen animations of the two types.

@doofus-01 doofus-01 merged commit 41b2030 into wesnoth:master Dec 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Graphics Issues that involve the graphics engine or assets. Units Issues that involve unit definitions or their implementation in the engine.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants