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: Apply billboard axis lock to right group #2323

Merged
merged 1 commit into from
Jan 27, 2025

Conversation

a-type
Copy link
Contributor

@a-type a-type commented Jan 24, 2025

Why

Somewhere along the way with Billboard, a wrapping Group was added, but some refs used to enforce axis lock were not updated to point to the inner group, which is the one being oriented.

What

Updated axis lock logic to utilize the inner group's ref

Checklist

  • Docs/Storybook describe intended behavior, no updates needed
  • Ready to be merged

Copy link

vercel bot commented Jan 24, 2025

@a-type is attempting to deploy a commit to the Poimandres Team on Vercel.

A member of the Team first needs to authorize it.

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.

@CodyJasonBennett CodyJasonBennett merged commit 39ef36b into pmndrs:master Jan 27, 2025
3 of 4 checks passed
Copy link

🎉 This PR is included in version 9.121.4 🎉

The release is available on:

Your semantic-release bot 📦🚀

@a-type a-type deleted the billboard-axis-lock branch January 27, 2025 14:42
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