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

[BUGFIX] Player's left notes being selected when they shouldn't #3093

Merged

Conversation

NotHyper-474
Copy link
Contributor

@NotHyper-474 NotHyper-474 commented Aug 5, 2024

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

Briefly describe the issue(s) fixed.

When selecting any part to the left of the chart any left note in the Player's side that is in the vertical boundary of the selection box would be mistakenly selected. This is better demonstrated in the before/after video below.

Include any relevant screenshots or videos.

fnf-bug-before-after.mp4

Copy link
Contributor

@Cartridge-Man Cartridge-Man left a comment

Choose a reason for hiding this comment

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

Thank god for this

@Hundrec
Copy link
Collaborator

Hundrec commented Aug 6, 2024

An amazingly simple fix!

@NotHyper-474 NotHyper-474 changed the title [BUGFIX] Player's left note being selected when it shouldn't [BUGFIX] Player's left notes being selected when they shouldn't Aug 6, 2024
@NotHyper-474
Copy link
Contributor Author

NotHyper-474 commented Aug 6, 2024

Just found a fix for this too. I don't like how it needs an additional else if check tho Actually I found something that doesn't require another check but nvm

2024-08-06.15-02-54.mp4

@Hundrec
Copy link
Collaborator

Hundrec commented Aug 6, 2024

Perhaps selecting events by highlighting any region on the right should be a quality of life feature! As it stands, it's a little difficult to select only a subset of events without accidentally picking up the player's right arrows as well.

@NotHyper-474
Copy link
Contributor Author

Perhaps selecting events by highlighting any region on the right should be a quality of life feature! As it stands, it's a little difficult to select only a subset of events without accidentally picking up the player's right arrows as well.

Makes sense, I guess I'll only keep the fix for the left part.

@NotHyper-474 NotHyper-474 changed the base branch from develop to main August 8, 2024 17:34
@NotHyper-474 NotHyper-474 changed the base branch from main to develop August 8, 2024 17:34
@EliteMasterEric EliteMasterEric added the status: pending triage Awaiting review. label Aug 14, 2024
@EliteMasterEric EliteMasterEric added type: minor bug Involves a minor bug or issue. topic: chart editor Related to the operation of the Chart Editor. status: accepted PR was approved for contribution. If it's not already merged, it may be merged on a private branch. and removed status: pending triage Awaiting review. labels Sep 22, 2024
@EliteMasterEric EliteMasterEric self-assigned this Sep 22, 2024
@EliteMasterEric EliteMasterEric added this to the 0.5.1 milestone Sep 22, 2024
@Hundrec
Copy link
Collaborator

Hundrec commented Sep 30, 2024

Finally in 0.5.1, woo!

@EliteMasterEric EliteMasterEric merged commit 7441c39 into FunkinCrew:develop Oct 1, 2024
@NotHyper-474 NotHyper-474 deleted the bugfix/chart-selection branch October 1, 2024 04:15
@Hundrec Hundrec added the size: small A small pull request with 10 or fewer changes. label Jan 22, 2025
@Hundrec Hundrec added the pr: haxe PR modifies game code. label Jan 22, 2025
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: small A small pull request with 10 or fewer changes. status: accepted PR was approved for contribution. If it's not already merged, it may be merged on a private branch. topic: chart editor Related to the operation of the Chart Editor. type: minor bug Involves a minor bug or issue.
Projects
None yet
8 participants