Skip to content

Fix(tooltip): allow non-ReactElement children#2079

Merged
jrgarciadev merged 2 commits into
heroui-inc:mainfrom
black197:fix/tooltip
Mar 5, 2024
Merged

Fix(tooltip): allow non-ReactElement children#2079
jrgarciadev merged 2 commits into
heroui-inc:mainfrom
black197:fix/tooltip

Conversation

@black197
Copy link
Copy Markdown
Contributor

@black197 black197 commented Dec 1, 2023

Closes #1840

📝 Description

Add a condition to check if children is valid ReactElement.

⛳️ Current behavior (updates)

When Tooltip gets string or number as children, they are not rendered.

🚀 New behavior

Now they will show on the screen normally.

💣 Is this a breaking change (Yes/No): No

📝 Additional Information

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Dec 1, 2023

🦋 Changeset detected

Latest commit: 09256c4

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 4 packages
Name Type
@nextui-org/tooltip Patch
@nextui-org/slider Patch
@nextui-org/snippet Patch
@nextui-org/react Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel
Copy link
Copy Markdown

vercel Bot commented Dec 1, 2023

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

Name Status Preview Comments Updated (UTC)
nextui-storybook-v2 ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 1, 2023 10:02am

@vercel
Copy link
Copy Markdown

vercel Bot commented Dec 1, 2023

@black197 is attempting to deploy a commit to the NextUI Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Copy Markdown
Member

@jrgarciadev jrgarciadev left a comment

Choose a reason for hiding this comment

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

Thank you! @black197

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.

[BUG] - Tooltip doesn't work inside Table

2 participants