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

Fixed Conflict With Modded StrumlineNote Sprite Looping Animation #3577

Merged
merged 3 commits into from
Oct 5, 2024

Conversation

DaWaterMalone
Copy link
Contributor

Does this PR close any issues? If so, link them below.

#3571

Briefly describe the issue(s) fixed.

The playStatic function on the StrumlineNote.hx file has this.active = false; which breaks modding compatibility for StrumlineNote sprites with looping animations. By setting this.active = true; it allows StrumlineNote sprites to loop an animation that is defined in the new notestyles folder.

Include any relevant screenshots or videos.

Broken StrumlineNote Animations when playStatic() this.active = false;

Broken.Animations.mp4

Working StrumlineNote Animations when setting playStatic() this.active = true;

Working.Animations.mp4

Custom NoteStyle JSON file for reference:
NoteStyle_JSON_Reference

Within the playStatic function on the StrumlineNote.hx, setting this.active = true; will fix looping animations for modded in StrumlineNote sprites. Leaving this.active = false; will stop any modded StrumlineNote sprite animation when the player or opponent hits chart notes.
@EliteMasterEric EliteMasterEric changed the base branch from main to develop October 5, 2024 16:45
@github-actions github-actions bot added the haxe Issue/PR modifies game code label Oct 5, 2024
@EliteMasterEric EliteMasterEric merged commit bc546e8 into FunkinCrew:develop Oct 5, 2024
2 checks passed
@github-actions github-actions bot added the medium A medium pull request with 100 or fewer changes label Oct 5, 2024
@EliteMasterEric EliteMasterEric added type: enhancement Provides an enhancement or new feature. mods Issue is related to the use of mods. status: accepted Approved for contribution. If it's not already merged, it may be merged on a private branch. labels Oct 5, 2024
@EliteMasterEric EliteMasterEric added this to the 0.5.2 milestone Oct 5, 2024
@EliteMasterEric
Copy link
Member

Merged after making a few tweaks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
haxe Issue/PR modifies game code medium A medium pull request with 100 or fewer changes mods Issue is related to the use of mods. status: accepted Approved for contribution. If it's not already merged, it may be merged on a private branch. type: enhancement Provides an enhancement or new feature.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants