Open brenoliradev opened 2 months ago
Latest commit: d3987e22386537746eb6676eb5086957bc760038
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
The latest updates on your projects. Learn more about Vercel for Git ↗︎
Name | Status | Preview | Comments | Updated (UTC) |
---|---|---|---|---|
flowbite-react | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Apr 30, 2024 11:14am |
flowbite-react-storybook | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Apr 30, 2024 11:14am |
This update enhances tooltip functionality by introducing control over the open state of tooltips, enabling them to be permanently open or closed. The changes include new examples in the UI documentation and adjustments to the tooltip components to support this feature.
File Path | Change Summary |
---|---|
src/docs/pages/TooltipsPage.tsx |
Added examples for always open/closed tooltips. |
src/lib/components/Floating/... |
Added isOpen prop to control the open state. |
src/lib/components/Tooltip/... |
Added isOpen boolean property to TooltipProps . |
isOpen
prop aligns with the request for interactive control over tooltips, enhancing user interaction.🐇 Oh hop and tweak, the code does peak, 🌟
With props anew, the tooltips speak.
Open or closed, they now can stay,
A rabbit's touch, makes UI sway.
🎉 Celebrate, for bugs we shoo,
With every line, the features grew!
packages/ui/src/components/Tooltip/Tooltip.tsx (3)
`18-18`: The addition of `isOpen` as an optional property in `TooltipProps` is well-implemented and aligns with the PR's objectives. --- `34-34`: The default assignment of `isOpen` to `null` and its subsequent usage in the `Tooltip` component are correctly implemented to support both controlled and uncontrolled modes. --- `49-49`: Passing the `isOpen` prop to the `Floating` component is correctly implemented, ensuring that the tooltip's visibility can be externally controlled as intended.packages/ui/src/components/Floating/Floating.tsx (3)
`46-46`: The addition of `isOpen` as an optional property in `FloatingProps` is well-implemented and aligns with the PR's objectives. --- `63-63`: The default assignment of `isOpen` to `null` in the `Floating` component is correctly implemented to support both controlled and uncontrolled modes. --- `69-73`: The logic to determine the `open` state in the `Floating` component is well-implemented, using `useMemo` to efficiently handle changes in the `isOpen` prop and support both controlled and uncontrolled modes.
it seems like ur branch base HEAD is way out of date, leading to massive conflicts.
Please sync ur fork with latest themesberg:main
and then rework/reopen this PR, its easier this way.
Yeah. I just noticed it yesterday.
I was using my outdated main (on my personal repo) instead of upstream main.
Just fixed it!
Hey,
I'm pretending to be more active again giving contributions to flow-bite. It has some time from my last contributions so I'm feeling a little ankward about the standards of this project lol
Just picked up #1257 and gave it a shot. To summarize, I'm adding the possibility to override the open/closed status for tooltips:
I implemented it in a simple way, just added a new prop called "isOpen" that overrides the "open" state - now called innerOpen. Since he also wanted to change it's trigger, I made it's default to "null" and did a simple check to ensure that it would work as expected when true or false.
Tbh idk if it's a valid way to do it here at flow-bite so let's discuss about it. :)
Cheers,
Summary by CodeRabbit
New Features
isOpen
property for the Floating and Tooltip components to control their visibility state.Documentation