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

Chart editor audio playback while dragging playheads fix #4140

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

Lasercar
Copy link
Contributor

@Lasercar Lasercar commented Feb 14, 2025

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

#3791

Briefly describe the issue(s) fixed.

Audio playback in the chart editor isn't automatically stopped while a playhead is dragged around - that, and you can start playback while dragging any of them around too.

This PR fixes this for all 3 playheads - the playbar, grid, and note preview.

Include any relevant screenshots or videos.

2025-02-15.12-06-15.mp4

@Lasercar Lasercar changed the base branch from main to develop February 14, 2025 14:16
@github-actions github-actions bot added status: pending triage Awaiting review. size: medium A medium pull request with 100 or fewer changes. pr: haxe PR modifies game code. and removed size: medium A medium pull request with 100 or fewer changes. labels Feb 14, 2025
@AbnormalPoof AbnormalPoof added type: minor bug Involves a minor bug or issue. topic: chart editor Related to the operation of the Chart Editor. size: medium A medium pull request with 100 or fewer changes. labels Feb 14, 2025
@amyspark-ng
Copy link
Contributor

you mean the minimap? it also does that (if the song is playing and you move it the song keeps replaying) i like it because it's funny but it's definely not intended

@Lasercar
Copy link
Contributor Author

Lasercar commented Feb 15, 2025

you mean the minimap? it also does that (if the song is playing and you move it the song keeps replaying) i like it because it's funny but it's definely not intended

Hmm, that doesn't appear to be the name of the uh, thing.

Wait, it's called the note preview playhead.

Fixed it! Yay!

Wow, it's just feels so clean now too!

This commit specifically fixes the note preview playhead.
@Lasercar Lasercar marked this pull request as ready for review February 15, 2025 02:04
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.

3 participants