Skip to content

Comments

Fix class select animation playback#1484

Merged
EricS-Valve merged 2 commits intoValveSoftware:masterfrom
k-vog:class-select-anims
Oct 23, 2025
Merged

Fix class select animation playback#1484
EricS-Valve merged 2 commits intoValveSoftware:masterfrom
k-vog:class-select-anims

Conversation

@k-vog
Copy link
Contributor

@k-vog k-vog commented Aug 8, 2025

This is a resubmission of #1285, which closed because of a deleted fork. I am the original author.

Hi. This fixes a bug where class select animations will freeze if you go back and forth in the menu.

Related issues:

Before

fix-class-select-anims-before.mp4

After

fix-class-select-anims-after.mp4

Details

When reusing the shared CTFPlayerModelPanel in the class select screen, SetupFlexWeights would incorrectly calculate a large negative m_flSceneTime. This would result in class select animations freezing when going back and forth in the class select menu.

-0.1s start times are allowed to give the system time to lerp to the looping animation. This constant is already used in neighboring code.

The 2nd commit just replaces this 0.1f magic number with a macro constant. This is optional, I just thought it would be a nice improvement.

k-vog added 2 commits August 8, 2025 16:37
When reusing the shared CTFPlayerModelPanel in the class select screen,
SetupFlexWeights would incorrectly calculate a large negative
m_flSceneTime. This would result in class select animations freezing
when going back and forth in the class select menu.

-0.1s start times are allowed to give the system time to lerp to the
looping animation. This constant is already used in neighboring code.
Bitl added a commit to BitlDevelopmentStudios/source-sdk-2013-bds-base that referenced this pull request Aug 10, 2025
@EricS-Valve EricS-Valve merged commit 38fd28f into ValveSoftware:master Oct 23, 2025
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

Successfully merging this pull request may close these issues.

2 participants