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

Implementation for Tooltip Component #120

Merged
merged 1 commit into from
Oct 25, 2023

Conversation

KushalNerella07
Copy link
Contributor

Dynamic Positioning: The tooltip supports top, bottom, left, and right positioning out of the box.
Responsive: The max-width of the tooltip adapts based on the container size, ensuring it looks great on all screens.
Theme Compliant: Fully integrated with the existing theme, using properties such as colors, spacings, and z-index layers.
Hover Interaction: The tooltip becomes visible on hover, providing a seamless experience for users.

Please note that I am aware that the I we cant fully see the tooltip for left, top and bottom and I think we need to make change to the storyboard in order to make it more visible. I am not aware on where to change it. Would love feedback on this addition and any potential improvements to make it even better! 🚀🔧

Dynamic Positioning: The tooltip supports top, bottom, left, and right positioning out of the box.
Responsive: The max-width of the tooltip adapts based on the container size, ensuring it looks great on all screens.
Theme Compliant: Fully integrated with the existing theme, using properties such as colors, spacings, and z-index layers.
Hover Interaction: The tooltip becomes visible on hover, providing a seamless experience for users.

Please note that I am aware that the I we cant fully see the tooltip for left, top and bottom and I think we need to make change to the storyboard in order to make it more visible. I am not aware on where to change it. Would love feedback on this addition and any potential improvements to make it even better! 🚀🔧
Copy link
Owner

@vczb vczb left a comment

Choose a reason for hiding this comment

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

LGTM

@vczb vczb merged commit 42f6dfb into vczb:main Oct 25, 2023
1 check passed
@KushalNerella07 KushalNerella07 deleted the feature/Tooltip branch October 31, 2023 16:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants