Skip to content

[BUGFIX] Prevent Story Mode props from resetting during fade - #7897

Merged
Hundrec merged 2 commits into
FunkinCrew:preview/public-playtestfrom
immalloy:fix/7865-story-prop-fade
Aug 21, 2026
Merged

[BUGFIX] Prevent Story Mode props from resetting during fade#7897
Hundrec merged 2 commits into
FunkinCrew:preview/public-playtestfrom
immalloy:fix/7865-story-prop-fade

Conversation

@immalloy

@immalloy immalloy commented Aug 8, 2026

Copy link
Copy Markdown

Linked Issues

Closes #7865

Description

When selecting a Story Mode level during player/BF prop's idle, the prop could resume its idle animation after the confirm animation finished while the screen was fading out.

The Story Menu now stops dispatching events to level props with a 'confirm' animation after a level has been selected. This prevents beat and update events from restarting prop animations during the transition, while allowing the confirmation to play normally.

Screenshots/Videos

Gif with the issue fixed Lmao

@github-actions github-actions Bot added pr: haxe PR modifies game code. status: pending triage Awaiting review. size: tiny A tiny pull request with 4 or fewer changes. labels Aug 8, 2026
@immalloy

immalloy commented Aug 8, 2026

Copy link
Copy Markdown
Author

why did the gif get all squisheddddd

@NotHyper-474 NotHyper-474 added the type: minor bug Involves a minor bug or issue. label Aug 8, 2026
@moxie-coder

Copy link
Copy Markdown
Contributor

why did the gif get all squisheddddd

I can see it fine

Prevent characters from restarting to their Idle animations post confirm in the story mode menu

Closes FunkinCrew#7865
@immalloy
immalloy force-pushed the fix/7865-story-prop-fade branch from 3619562 to 69c303f Compare August 8, 2026 21:51
@immalloy

immalloy commented Aug 8, 2026

Copy link
Copy Markdown
Author

updated the commit to contain a fix so non confirm props can play animations even in transitions
thanks to @ComedyLost for this suggestion

@Hundrec Hundrec linked an issue Aug 10, 2026 that may be closed by this pull request
4 tasks

@EliteMasterEric EliteMasterEric left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Check the review comment I left.

Comment thread source/funkin/ui/story/StoryMenuState.hx
@immalloy

Copy link
Copy Markdown
Author

Fixed in the latest commit!! i removed the lil guard above for it

@immalloy

Copy link
Copy Markdown
Author

non-confirm props still receive events during the transition.

@EliteMasterEric EliteMasterEric added status: reviewing internally Under consideration and testing. status: accepted PR was approved for contribution. If it's not already merged, it may be merged on a private branch. and removed status: pending triage Awaiting review. status: reviewing internally Under consideration and testing. labels Aug 18, 2026
@AbnormalPoof
AbnormalPoof force-pushed the preview/public-playtest branch from 401b06b to 2e7f104 Compare August 20, 2026 03:25
@JackXson-Real

Copy link
Copy Markdown
Contributor

Not Eric accepting a PR that will not even be seen in a patch because 0.8.6 is the last patch before the menu revamps 😭

@immalloy

Copy link
Copy Markdown
Author

😔😔😔😔

@Hundrec Hundrec added this to the 0.9.0 - Week 8 milestone Aug 21, 2026
@Hundrec
Hundrec merged commit 6cc2f27 into FunkinCrew:preview/public-playtest Aug 21, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr: haxe PR modifies game code. size: tiny A tiny pull request with 4 or fewer changes. status: accepted PR was approved for contribution. If it's not already merged, it may be merged on a private branch. type: minor bug Involves a minor bug or issue.

Development

Successfully merging this pull request may close these issues.

Bug Report: Story Mode player prop can reset animation mid fade

6 participants