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/ENHANCEMENT] Changes instances of FlxG.mouse.screen(x/y) to FlxG.mouse.view(x/y) in ChartEditorState.hx (and more) #3337

Merged
merged 4 commits into from
Sep 30, 2024

Conversation

Kade-github
Copy link
Contributor

@Kade-github Kade-github commented Sep 17, 2024

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

Fixes #3147,

Briefly describe the issue(s) fixed.

Changes any instance of screenX/screenY to viewX/viewY as the former is depreciated. Causing the editor to not function due to HaxeFlixel/flixel#3210

Also adds a millisecond part of the playbar :D

Include any relevant screenshots or videos.

image

remove laggy redundant traces

Revert "remove laggy redundant traces"

This reverts commit 2041138.
@EliteMasterEric EliteMasterEric self-assigned this Sep 17, 2024
@EliteMasterEric EliteMasterEric added type: major bug Involves a major bug, including crashes, softlocks, or issues blocking progression status: reviewing internally This PR is under internal review and quality assurance testing chart editor Issue is related to the operation of the Chart Editor. labels Sep 17, 2024
@Kade-github Kade-github changed the title [ENHANCEMENT] Changes instance of FlxG.mouse.screen(x/y) to FlxG.mouse.view(x/y) in ChartEditorState.hx (and more) [ENHANCEMENT] Changes instances of FlxG.mouse.screen(x/y) to FlxG.mouse.view(x/y) in ChartEditorState.hx (and more) Sep 17, 2024
@Kade-github Kade-github changed the title [ENHANCEMENT] Changes instances of FlxG.mouse.screen(x/y) to FlxG.mouse.view(x/y) in ChartEditorState.hx (and more) [BUGFIX/ENHANCEMENT] Changes instances of FlxG.mouse.screen(x/y) to FlxG.mouse.view(x/y) in ChartEditorState.hx (and more) Sep 17, 2024
Copy link
Contributor

@ImCodist ImCodist left a comment

Choose a reason for hiding this comment

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

nice changes!

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.

LGTM

@EliteMasterEric EliteMasterEric added this to the 0.5.1 milestone Sep 17, 2024
@EliteMasterEric EliteMasterEric added status: accepted Approved for contribution. If it's not already merged, it may be merged on a private branch. and removed status: reviewing internally This PR is under internal review and quality assurance testing labels Sep 17, 2024
Hazardous2468 added a commit to Hazardous2468/Funkin_Hazard-Modcharting-Fork that referenced this pull request Sep 18, 2024
@EliteMasterEric EliteMasterEric changed the base branch from main to develop September 25, 2024 12:56
@ninjamuffin99 ninjamuffin99 merged commit 9624b57 into FunkinCrew:develop Sep 30, 2024
@tposejank
Copy link
Contributor

Yay

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chart editor Issue is related to the operation of the Chart Editor. status: accepted Approved for contribution. If it's not already merged, it may be merged on a private branch. type: major bug Involves a major bug, including crashes, softlocks, or issues blocking progression
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug Report: Chart Editor is completely broken (does not allow placing notes)