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 no minimum for event values fix #4109

Conversation

Lasercar
Copy link
Contributor

@Lasercar Lasercar commented Feb 6, 2025

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

#2861

Briefly describe the issue(s) fixed.

The default events didn't have minimum values set, lol.

Include any relevant screenshots or videos.

2025-02-06.18-11-41.1.mp4

EliteMasterEric and others added 20 commits January 16, 2025 19:50
* Rewrite CHANGELOG.md for v0.5.2

* [docs] Add missing changes and links to Changelog (FunkinCrew#3655)

* changelog sweep

* Resolve a few conflicts

* Resolve more conflicts

* DadBattle

* Re-add @ianharrigan

* Add links to 0.5.0

* Add links and contributors to 0.4.1 and 0.4.0

* Add links to 0.3.3 and finishing touches

* Add and polish 0.5.3 changes

* Make spacing consistent

* Make label actions more convenient (FunkinCrew#3771)

* fixed a plethora of dead links (FunkinCrew#3831)

* Add `status: planned` label to label actions (FunkinCrew#3910)

* Make label actions more convenient

* chore: Add status: planned to label actions

* Bump the Stage data version to indicate that new values were added.

* Updated Save data version to indicate new values.

* Only display the Stage Editor in builds that enable it, and enable it by default (this re-enables the Stage Editor keybind).

* Enable log messages on all builds by default after we got too many complaints.

* Remove unused custom type resolver.

* Update save data format and error handling.

* Fix a typo.

* proj hxp version bump

* changelog yoink

* Remove duplicate issue detector. (FunkinCrew#3944)

---------

Co-authored-by: Hundrec <[email protected]>
Co-authored-by: Abnormal <[email protected]>
Co-authored-by: Cameron Taylor <[email protected]>
…3721)

* add events for focus gain/lost

* add callbacks

* move the events to a new interface

* Ok NOW it works

* the event can now longer be canceled
* add album title offsets

* cleanup

* cleanup

* fix inconsistency
* Custom render distance strumline

* Custom note positions

* Custom vertices for holds

* Song Retry Event

Song retry event for pausesubstate.hx
Song retry event for playstate.hx
Song retry event for stage.hx
Song retry event for bopper.hx
Song retry event for song.hx
Scripted class event
Module event

* this should be cast

* Show notesplash toggle

* Formatting and EOF
lol, lmao, imagine coding a input to have minimum values but just.. not setting a value?
@Lasercar
Copy link
Contributor Author

Lasercar commented Feb 6, 2025

oh, shit

@github-actions github-actions bot added status: pending triage Awaiting review. size: large A large pull request with more than 100 changes. pr: haxe PR modifies game code. and removed size: large A large pull request with more than 100 changes. labels Feb 6, 2025
@Lasercar Lasercar closed this Feb 6, 2025
@Lasercar Lasercar deleted the chart-editor-no-min-for-event-values-fix branch February 6, 2025 08:20
@Lasercar
Copy link
Contributor Author

Lasercar commented Feb 6, 2025

How the heck did that happen? And how do I fix this?

@Lasercar
Copy link
Contributor Author

Lasercar commented Feb 6, 2025

oh, wait
I'm being an idiot again.

@AbnormalPoof AbnormalPoof removed the status: pending triage Awaiting review. label Feb 6, 2025
@AbnormalPoof AbnormalPoof added status: rejected Issue did not pass review or PR cannot be approved. and removed pr: haxe PR modifies game code. labels Feb 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: rejected Issue did not pass review or PR cannot be approved.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants