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

Hold note still pressed after pausing, removing hold and resuming game fix #4145

Draft
wants to merge 7 commits into
base: develop
Choose a base branch
from

Conversation

Lasercar
Copy link
Contributor

@Lasercar Lasercar commented Feb 15, 2025

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

Fixes #2227

Briefly describe the issue(s) fixed.

Pressing a hold note then pausing the game without unpressing, then releasing your hold and resuming, results in the hold note still being pressed despite having removed your input.

This fixes it for a single hold note, but it probably doesn't work properly on multiple hold notes? I haven't tested it on multiple hold notes.

Include any relevant screenshots or videos.

2025-02-15.15-26-26.mp4

Whoops, I didn't rebuild after making the branch, lol. Ah whatever, who cares about that?

Weird issue - hold note being dropped after pressing alternative note inputs
2025-02-18.21-12-28.mp4

@Lasercar Lasercar changed the base branch from main to develop February 15, 2025 05:24
@github-actions github-actions bot added pr: haxe PR modifies game code. size: small A small pull request with 10 or fewer changes. status: pending triage Awaiting review. labels Feb 15, 2025
@Lasercar Lasercar changed the title Hold note still pressed after resuming game fix Hold note still pressed after pausing, removing hold and resuming game fix Feb 15, 2025
@AbnormalPoof AbnormalPoof added the type: minor bug Involves a minor bug or issue. label Feb 15, 2025
Copy link
Collaborator

@AbnormalPoof AbnormalPoof left a comment

Choose a reason for hiding this comment

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

LGTM

@EliteMasterEric EliteMasterEric added status: needs revision Cannot be approved because it is awaiting some work by the contributor. and removed status: pending triage Awaiting review. labels Feb 18, 2025
@EliteMasterEric
Copy link
Member

Definitely test and make sure it works on multiple hold notes before requesting review again.

@Lasercar
Copy link
Contributor Author

Lasercar commented Feb 18, 2025

Definitely test and make sure it works on multiple hold notes before requesting review again.

I've just tested it on multiple hold notes and it definitely works, mostly.

I've noticed one thing where sometimes, it works for one hold note, but not the other one?

@Hundrec I've just uploaded it to the description.

I'll see if I can get a clip of the weird issue I mentioned.

@Hundrec
Copy link
Collaborator

Hundrec commented Feb 18, 2025

I think that's what Eric meant

Can you record and send a video of the current behavior?

@Lasercar
Copy link
Contributor Author

Lasercar commented Feb 18, 2025

Ah, I think it's because of the alternative note inputs - notice how the down arrow hold doesn't work, after I pressed down a few times?

But then again, maybe it's just a me problem? nope.

@Hundrec
Copy link
Collaborator

Hundrec commented Feb 18, 2025

In your videos, are you still holding the note while unpausing?

The behavior in the videos don't seem any different from 0.5.3

If so, can you record a video of Bopeebo without holding the note while unpausing?

@Lasercar
Copy link
Contributor Author

Lasercar commented Feb 18, 2025

In your videos, are you still holding the note while unpausing?

The behavior in the videos don't seem any different from 0.5.3

If so, can you record a video of Bopeebo without holding the note while unpausing?

Ah, yeah, I see another problem when I pause while holding, unhold, and then resume:

2025-02-18.18-49-27.mp4

Sometimes, a note hold still sticks?

I managed to fix this issue, but I'm still getting the other one mentioned above: #4145 (comment)

Where the first alternative input to be pressed causes that note to be dropped?

@Hundrec
Copy link
Collaborator

Hundrec commented Feb 18, 2025

Yes, I believe this is what Eric meant.
Please mark this PR as a draft for now since it doesn't seem to fix the notes staying held

@Lasercar Lasercar marked this pull request as draft February 18, 2025 08:52
Note hold being dropped after hitting alternative note button in pause screen
@github-actions github-actions bot added size: medium A medium pull request with 100 or fewer changes. and removed size: small A small pull request with 10 or fewer changes. labels Feb 18, 2025
Note hold being dropped after hitting alternative note button in pause screen
@Lasercar Lasercar force-pushed the Hold-note-still-pressed-after-resuming-game-fix branch from fdacd10 to 0e6f04a Compare February 18, 2025 10:59
Sticking hold note
@Lasercar Lasercar marked this pull request as ready for review February 18, 2025 11:07
@Lasercar Lasercar marked this pull request as draft February 18, 2025 11:11
causes the first hold note that gets alt pressed to be dropped
@Lasercar Lasercar force-pushed the Hold-note-still-pressed-after-resuming-game-fix branch from 6067e60 to fee02a2 Compare February 18, 2025 13:45
@Lasercar
Copy link
Contributor Author

Lasercar commented Feb 18, 2025

I think this is going to remain a draft for a while - I basically need a way to check if one of the keys is being held down and return if so but it turns out Just_Pressed and Pressed end up true at the same time (aka the first frame) when the key is pressed, so there's no way I can fix this issue atm:
https://haxeflixel.com/documentation/keyboard/

If there was another filter called Hold or something that only returns true after one frame has passed of the key being down, then I'd be able to fix the remaining issue.

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: needs revision Cannot be approved because it is awaiting some work by the contributor. type: minor bug Involves a minor bug or issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants