Skip to content

Touch to display tooltip (Not long touch) #4206

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

Open
Steven-MKN opened this issue Nov 29, 2023 · 3 comments
Open

Touch to display tooltip (Not long touch) #4206

Steven-MKN opened this issue Nov 29, 2023 · 3 comments

Comments

@Steven-MKN
Copy link

Is your feature request related to a problem? Please describe.
I have an info icon like ℹ️ or similar. I would like my users to touch the icon, without long touch, as this would be more intuitive

Describe the solution you'd like
Can we add an option to touch without to long touch for the tooltip text to be triggered? I'd be happy to pick this up

Describe alternatives you've considered
I have tried setting the enterTouchDelay prop to something like 0 or 1 so that it might trigger much sooner. Seems like this delay is only applicable after a long touch is registered

Additional context
Screenshot 2023-11-29 at 04 57 31
an example where I'd prefer a touch instead of a long touch

@Jhh910
Copy link

Jhh910 commented Jun 26, 2024

Is your feature request related to a problem? Please describe. I have an info icon like ℹ️ or similar. I would like my users to touch the icon, without long touch, as this would be more intuitive

Describe the solution you'd like Can we add an option to touch without to long touch for the tooltip text to be triggered? I'd be happy to pick this up

Describe alternatives you've considered I have tried setting the enterTouchDelay prop to something like 0 or 1 so that it might trigger much sooner. Seems like this delay is only applicable after a long touch is registered

Additional context Screenshot 2023-11-29 at 04 57 31 an example where I'd prefer a touch instead of a long touch

Did you solve it? I need that, too

@Steven-MKN
Copy link
Author

@Jhh910 You could use Patch Package to update the module for yourself. But I didn't end up fixing this. I'll create a PR for this, hopefully it gets accepted (Material design docs doesn't mention this usecase, and we seem to be driven heavily by it, but I'll give it a shot)

@Steven-MKN
Copy link
Author

Made a pr here #4479

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants