Skip to content

feat: initial implementation for curve editor #51

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.

Sign up for GitHub

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

Jump to bottom
Open
Tnixc wants to merge 13 commits into
base: main
Choose a base branch
from
Open

feat: initial implementation for curve editor #51

Tnixc wants to merge 13 commits into from

Conversation

@Tnixc
Copy link

@Tnixc Tnixc commented Sep 20, 2025

image

@wang-edward
Copy link
Collaborator

wang-edward commented Sep 22, 2025 •
edited
Loading

Looks really good but 1k lines is pretty hard to review
Can you try to cut it down to like 500?

We need to be able to understand what every single line of code
Maybe use chat gpt to refactor

@Tnixc
Copy link
Author

Tnixc commented Sep 23, 2025

Okay I tried that as much as I can, still, most of it is functional

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Tnixc @wang-edward