-
Notifications
You must be signed in to change notification settings - Fork 27
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(core)!: Implement floating UI for tooltip positioning and styling #108
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good so far! There're some code reformats that are making it hard to review precisely what has changed. I left a few suggestions as well 🙂
package/src/lib/components/tour-overlay/shapes/CircleShape.component.tsx
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is looking great! I left a few suggestions and comments. Please address them before merging. Thank you @alejo0o 🎉
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, awesome change! Thanks for your help @alejo0o 🎉
#108) BREAKING CHANGE: Refactored how tooltips can be positioned relative to the spot using FloatingUI. This change replaces the `alignTo` and `position` props of the SpotlightTourProvider and TourStep with `floatingProps`. The new prop lets you configure the behavior of FloatingUI for your tooltips
This PR adds Floating UI in order to handle tooltip position which was having issues on IPad screens. This implementation delegates alignment and positioning to floating UI this way the user can configure floating library for each tourstep or provide a global configuration in order to show the tooltip in different positions, provide offsets, add floating middlewares,etc.