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

[Tooltip][Mobile] - Not triggering on focus #1351

Closed
alinebujato opened this issue Apr 28, 2022 · 6 comments
Closed

[Tooltip][Mobile] - Not triggering on focus #1351

alinebujato opened this issue Apr 28, 2022 · 6 comments

Comments

@alinebujato
Copy link

Bug report

Current Behavior

The tooltip is not triggered when focusing over the trigger on mobile. This is expected accessibility behavior.

Expected behavior

The tooltip should show on focus.

Reproducible example

Tooltip - Mobile

Suggested solution

Additional context

Your environment

Software Name(s) Version
Radix Package(s) @radix-ui/react-tooltip 0.1.7
React n/a
Browser all
Assistive tech
Node n/a
npm/yarn
Operating System all
@drianoaz
Copy link

I did some tests and apparently this problem was solved in the version 0.1.8-rc.10

@andy-hook
Copy link
Collaborator

This is by design #955 (comment)

I did some tests and apparently this problem was solved in the version 0.1.8-rc.10

@benoitgrelard Given the previous point, is this is a regression from this change?

@benoitgrelard
Copy link
Collaborator

Yeah so it's definitely by design and they shouldn't open on mobile.

@benoitgrelard Given the previous point, is this is a regression from this change?

Hmm, yeah potentially. I'll close this issue be open another one for the regression 👍

@benoitgrelard
Copy link
Collaborator

#1354

@Simonpedro
Copy link

Yeah so it's definitely by design and they shouldn't open on mobile.

Hey @benoitgrelard, I just noticed it doesn't open on mobile and thought it was a bug or a misconfiguration and ended up here. Out of curiosity, can I ask what's the rationale behind this statement? Thanks!

@benoitgrelard
Copy link
Collaborator

Hey @Simonpedro,

Have a look here for some more explanations: #955

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

No branches or pull requests

5 participants