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

feat: Tooltip for Buttons #481

Merged
merged 3 commits into from Jun 14, 2023

Conversation

gitstart
Copy link
Contributor

@gitstart gitstart commented May 24, 2023

Fixes: #471
What: Add the possibility to use tooltips on button component
Why: To be able to use tooltips in this component
How: The changes was implemented following the components src/components/icon-elements/Icon/Icon.tsx and src/components/icon-elements/Badge/Badge.tsx, they already have a tooltip component

Testing

run:

npm run storybook
  1. Go to InputElement/Button/Sizes
  2. Hover over the first column buttons

Demo

Loom Video: https://www.loom.com/share/5181bda86a65435b8f6298ba92dfb493


This code was written and reviewed by GitStart Community. Growing great engineers, one PR at a time.

@gitstart gitstart changed the title TRMR-OSS-20 - Enhancement Request: Tooltip for buttons Enhancement Request: Tooltip for buttons May 24, 2023
@gitstart gitstart marked this pull request as ready for review May 24, 2023 16:30
@gitstart gitstart marked this pull request as draft May 24, 2023 16:30
@gitstart gitstart force-pushed the feat/add-tooltip-for-buttons branch 2 times, most recently from 52b34f8 to e2150d1 Compare May 24, 2023 16:33
@gitstart gitstart changed the title Enhancement Request: Tooltip for buttons feat: Tooltip for Buttons May 24, 2023
@gitstart gitstart marked this pull request as ready for review May 24, 2023 16:36
@gitstart
Copy link
Contributor Author

👋 Hello @mitrotasios! Can you review this PR, please?

@mitrotasios mitrotasios added the Type: Feature New feature for existing component label May 26, 2023
Copy link
Member

@mitrotasios mitrotasios left a comment

Choose a reason for hiding this comment

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

Great job! Would it be possible to add a small delay to the tooltip (e.g. 150-300ms)? I think this is better for tooltips on buttons. :)

@gitstart
Copy link
Contributor Author

Great job! Would it be possible to add a small delay to the tooltip (e.g. 150-300ms)? I think this is better for tooltips on buttons. :)

we will work on it and get back to you soon.

@gitstart
Copy link
Contributor Author

gitstart commented Jun 1, 2023

Hello @mitrotasios
Changes have been made, you can see a tooltip speed difference on an icon and a button, in the video in the description

@gitstart gitstart requested a review from mitrotasios June 6, 2023 02:12
@gitstart gitstart force-pushed the feat/add-tooltip-for-buttons branch from 0cdfe1e to 7c19d9e Compare June 12, 2023 20:29
Co-authored-by: Rafael Toledo <87545086+Toledodev@users.noreply.github.com>
Co-authored-by: Nitesh Singh <nitesh.singh@gitstart.dev>
Co-authored-by: Rubens Rafael <70234898+RubensRafael@users.noreply.github.com>
@gitstart gitstart force-pushed the feat/add-tooltip-for-buttons branch from 7c19d9e to 7522196 Compare June 12, 2023 20:30
Copy link
Member

@mitrotasios mitrotasios left a comment

Choose a reason for hiding this comment

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

LGTM!

@mitrotasios mitrotasios merged commit 4c4e918 into tremorlabs:main Jun 14, 2023
3 checks passed
@github-actions
Copy link

🎉 This PR is included in version 3.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@abdullah-txfusion
Copy link

Seems like there's no straightforward way to style the tooltip

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Feature New feature for existing component
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Enhancement Request: Tooltip for buttons
3 participants