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

feat: support for tweakpane v4 #2

Merged
merged 2 commits into from
Nov 9, 2023
Merged

Conversation

kitschpatrol
Copy link
Contributor

@kitschpatrol kitschpatrol commented Sep 21, 2023

Thanks for this plugin.

I ported it to Tweakpane 4.x. This is a breaking change for users of Tweakpane 3.x.

The updates were mostly based on the following documentation:

I made some minor structural changes to more closely reflect the tweakpane plugin-template.

@0b5vr
Copy link
Owner

0b5vr commented Nov 9, 2023

Thank you for the PR! will look into it

@0b5vr
Copy link
Owner

0b5vr commented Nov 9, 2023

I wondered why eslint has to be bumped but

I made some minor structural changes to more closely reflect the tweakpane plugin-template.

It's very nice of you. 🥰

package.json Outdated Show resolved Hide resolved
Co-authored-by: 0b5vr <0b5vr@0b5vr.com>
@@ -88,6 +86,6 @@ RotationInputPluginQuaternionParams
pickerLayout: picker ?? 'popup',
value,
viewProps: viewProps,
} );
} ) as unknown as ValueController<Quaternion>; // TODO;
Copy link
Owner

Choose a reason for hiding this comment

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

nice TODO. I couldn't come up with a viable solution.

Copy link
Owner

@0b5vr 0b5vr left a comment

Choose a reason for hiding this comment

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

Looks PERFECT. Thanks a lot!!

@0b5vr 0b5vr merged commit 9fbc443 into 0b5vr:dev Nov 9, 2023
@kitschpatrol
Copy link
Contributor Author

Thanks for sharing your work!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants