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

bug(material/tooltip): fix for tooltip created a new problem #26557

Closed
1 task done
ewolfman opened this issue Feb 2, 2023 · 2 comments · Fixed by #26786
Closed
1 task done

bug(material/tooltip): fix for tooltip created a new problem #26557

ewolfman opened this issue Feb 2, 2023 · 2 comments · Fixed by #26786
Assignees
Labels
area: material/tooltip P4 A relatively minor issue that is not relevant to core functions

Comments

@ewolfman
Copy link

ewolfman commented Feb 2, 2023

Is this a regression?

  • Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was

13.2.4

Description

Hi,

I think that this fix: #24475 created a new issue:
When the tooltip is showing you can move the mouse to it (nice), however when you move the mouse even further away, the tooltip should having been closed. But the current behavior is that the tooltip remains until you leave the margin area (I think), causing the transparent margin to 'hide' controls beneath the tooltip and they cannot be reached using the mouse.

Reproduction

See this example: https://stackblitz.com/edit/angular-tmb9al-4kaa4l?file=src/app/tooltip-delay-example.html

  1. Hover over button and tooltip shows.
  2. Try clicking on the input field while the tooltip is showing.

Expected Behavior

Tooltip should not block focusing on elements (behavior as it was before 13.2.5).
Same demo with 13.2.4: https://stackblitz.com/edit/angular-tmb9al-1wtpjv?file=package.json,src%2Findex.html,package-lock.json,src%2Fapp%2Ftooltip-delay-example.html

13.2.4.mp4
13.2.5.mp4

Actual Behavior

Cannot click the input field.

Environment

  • Angular: 13.2.5
  • CDK/Material: 13.2.5
  • Browser(s): Edge, Firefox
@ewolfman ewolfman added the needs triage This issue needs to be triaged by the team label Feb 2, 2023
@crisbeto crisbeto self-assigned this Mar 14, 2023
@crisbeto crisbeto added P4 A relatively minor issue that is not relevant to core functions area: material/tooltip and removed needs triage This issue needs to be triaged by the team labels Mar 14, 2023
crisbeto added a commit to crisbeto/material2 that referenced this issue Mar 14, 2023
Fixes that we were adding an additional hover area on all sides of the tooltip, rather than only the side that is next to the trigger.

Fixes angular#26557.
crisbeto added a commit that referenced this issue Mar 20, 2023
…ger (#26786)

Fixes that we were adding an additional hover area on all sides of the tooltip, rather than only the side that is next to the trigger.

Fixes #26557.
crisbeto added a commit that referenced this issue Mar 20, 2023
…ger (#26786)

Fixes that we were adding an additional hover area on all sides of the tooltip, rather than only the side that is next to the trigger.

Fixes #26557.

(cherry picked from commit 142e845)
@eddyuk
Copy link

eddyuk commented Mar 30, 2023

Can this also be fixed in legacy components please?

@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Apr 30, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area: material/tooltip P4 A relatively minor issue that is not relevant to core functions
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants