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

fix: missing style prop from type in ScrollControls #1766

Merged
merged 1 commit into from
Dec 26, 2023

Conversation

bhavishya-sahdev
Copy link
Contributor

@bhavishya-sahdev bhavishya-sahdev commented Dec 26, 2023

Why

Currently, when using the Scroll component with the html prop, the style prop is missing in the prop list. So to fix the missing auto-complete, I made the change in question.
image

What

Updated the ScrollProps type to better fit the current needs. The style prop will be optional when the html is passed and it will not be accepted without the html prop.

without passing html prop:
image

with html prop:
image

Checklist

  • Ready to be merged

Copy link

vercel bot commented Dec 26, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
drei ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 26, 2023 6:54am

Copy link

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit acbe3c4:

Sandbox Source
xenodochial-cdn-gv6dr8 Configuration
Ground reflections and video textures Configuration
arc-x-pmndrs-colors Configuration

@CodyJasonBennett CodyJasonBennett merged commit 639fca1 into pmndrs:master Dec 26, 2023
3 checks passed
Copy link

🎉 This PR is included in version 9.92.7 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

2 participants