Skip to content

[BUGFIX] Chart Editor - Variation fallback & chartdata only difficulties in difficulty toolbox #5215

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

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

Lasercar
Copy link
Contributor

@Lasercar Lasercar commented Jun 9, 2025

Linked Issues

Fixes #5206 (For cases when a chartdata difficulty isn't in the metadata)

Description

Fixes the variation fallback - if there's no default variation then it'll use the first available variation.

Difficulties that exist in the chartdata but are missing in metadata for that variation now appear in the difficulty toolbox. They aren't added to the metadata, however.

Screenshots/Videos

2025-06-09.21-28-58.mp4

@Lasercar Lasercar changed the base branch from main to develop June 9, 2025 11:36
@github-actions github-actions bot added pr: haxe PR modifies game code. status: pending triage Awaiting review. size: medium A medium pull request with 100 or fewer changes. labels Jun 9, 2025
@Lasercar Lasercar marked this pull request as draft June 9, 2025 11:38
@Lasercar Lasercar force-pushed the bugfix/chart-editor-add-missing-chartdata-difficulties-to-metadata branch from 441fed5 to f8c518f Compare June 9, 2025 11:42
@Lasercar Lasercar marked this pull request as ready for review June 9, 2025 11:56
@Lasercar
Copy link
Contributor Author

Lasercar commented Jun 9, 2025

One thing of note, since this adds missing chartdata difficulties to the metadata difficulties for that variation, this means that stuff like picospeaker will show up in the difficulties, and it'll be saved as a playable difficulty?

I think instead I'll just make it add the difficulties to the difficulty toolbox instead, and later, I can add a button to add/remove said difficulty from the metadata, and add some kind of visual in the metadata toolbox.

Until that stuff, there'll be a few cases I'm sure of someone complaining why their difficulty shows up in the chart editor but not in freeplay or whatever, because the chart editor does not indicate that the difficulty isn't in the metadata.

@Lasercar Lasercar force-pushed the bugfix/chart-editor-add-missing-chartdata-difficulties-to-metadata branch 2 times, most recently from 1b295aa to 0e68bbf Compare June 9, 2025 12:55
@Lasercar Lasercar changed the title [BUGFIX] Chart Editor - Variation fallback & add missing chartdata difficulties to metadata [BUGFIX] Chart Editor - Variation fallback & add chartdata only difficulties to difficulty toolbox Jun 9, 2025
@AbnormalPoof AbnormalPoof added type: minor bug Involves a minor bug or issue. topic: chart editor Related to the operation of the Chart Editor. labels Jun 9, 2025
@Lasercar Lasercar force-pushed the bugfix/chart-editor-add-missing-chartdata-difficulties-to-metadata branch from 0e68bbf to 346515e Compare June 11, 2025 03:17
@Lasercar Lasercar changed the title [BUGFIX] Chart Editor - Variation fallback & add chartdata only difficulties to difficulty toolbox [BUGFIX] Chart Editor - Variation fallback & chartdata only difficulties in difficulty toolbox Jun 11, 2025
@Lasercar Lasercar force-pushed the bugfix/chart-editor-add-missing-chartdata-difficulties-to-metadata branch from 346515e to 30f6bb9 Compare June 17, 2025 11:52
@Lasercar Lasercar force-pushed the bugfix/chart-editor-add-missing-chartdata-difficulties-to-metadata branch from 30f6bb9 to b4d0951 Compare July 22, 2025 14:08
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: medium A medium pull request with 100 or fewer changes. status: pending triage Awaiting review. topic: chart editor Related to the operation of the Chart Editor. type: minor bug Involves a minor bug or issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug Report: Disappearing notes/events when switching songs in the Chart Editor
2 participants