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

[FEATURE] Custom Note Style Support #2581

Merged

Conversation

Keoiki
Copy link
Contributor

@Keoiki Keoiki commented May 21, 2024

The Funkin' devs did an oopsie and hardcoded note styles in songs and the chart editor.

  • Songs now use the note style provided in the respective variation's metadata file.
  • The Note Styles dropdown menu in the Chart Editor now properly populates with entries from the NoteStyleRegistery. (It had a function all along but they forgot to use it lol)

Consired merging FunkinCrew/funkin.assets#8 into the assets repo as it fixes Week 6 Erect songs having the wrong note style as a result of this change.

@EliteMasterEric EliteMasterEric self-assigned this May 21, 2024
@EliteMasterEric EliteMasterEric changed the title Note Style softcoding [FEATURE] Custom Note Style Support May 22, 2024
Copy link
Member

@EliteMasterEric EliteMasterEric left a comment

Choose a reason for hiding this comment

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

Works great when I tested!

@EliteMasterEric EliteMasterEric merged commit b6724eb into FunkinCrew:develop May 24, 2024
@Keoiki Keoiki deleted the playstate-notestyles-query branch June 4, 2024 23:24
@gamerbross
Copy link
Contributor

FunkinCrew/funkin.assets#8 didn't got merged and now the week 6 erects note styles are broken 😭

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.

3 participants