Skip to content

Fixed stutter once a wipe transition to the Main Menu completes.#2315

Merged
ninjamuffin99 merged 1 commit intoFunkinCrew:developfrom
ImCodist:bugfix/main-menu-camera-stutter
May 9, 2024
Merged

Fixed stutter once a wipe transition to the Main Menu completes.#2315
ninjamuffin99 merged 1 commit intoFunkinCrew:developfrom
ImCodist:bugfix/main-menu-camera-stutter

Conversation

@ImCodist
Copy link
Contributor

@ImCodist ImCodist commented May 8, 2024

There is this bit of code in the MainMenuState that I assume was supposed to run immediately after changing to the state but instead is run after the transition completes to the state.
This causes the camera to be reset when the transition completes, creating a stutter effect.

I've tested this from multiple scenarios where the state is changed to MainMenuState and nothing seems to break.

i am not sure why this bit of code exists in the first place actually since the camera reset call made just before it seems to work just fine but idk

Footage of this problem happening in game (i apologize for the overlapping music i am too tired to rerecord this clip):

Funkin_JsBtvCm3OO.mp4

@ninjamuffin99
Copy link
Member

haha wow nice catch! that bugged me for the longest time lol....

@ninjamuffin99 ninjamuffin99 merged commit 4bd8542 into FunkinCrew:develop May 9, 2024
@ninjamuffin99 ninjamuffin99 added this to the 0.3.3 milestone May 9, 2024
@ImCodist ImCodist deleted the bugfix/main-menu-camera-stutter branch May 15, 2024 02:57
@Hundrec Hundrec added status: accepted PR was approved for contribution. If it's not already merged, it may be merged on a private branch. type: optimization Involves a performance issue or a bug which causes lag. size: small A small pull request with 10 or fewer changes. pr: haxe PR modifies game code. labels Jan 22, 2025
@EliteMasterEric EliteMasterEric added size: tiny A tiny pull request with 4 or fewer changes. and removed size: small A small pull request with 10 or fewer changes. labels May 14, 2025
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: optimization Involves a performance issue or a bug which causes lag.

Development

Successfully merging this pull request may close these issues.

4 participants