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.

Sign up for GitHub

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

Jump to bottom

HvTooltip displayed after component page unmounts #3411

Closed
2 tasks done
stephaniegaspar opened this issue Jun 6, 2023 · 1 comment
Closed
2 tasks done

HvTooltip displayed after component page unmounts #3411

stephaniegaspar opened this issue Jun 6, 2023 · 1 comment
Labels
🐛 bug Something isn't working

Comments

@stephaniegaspar
Copy link
Contributor

stephaniegaspar commented Jun 6, 2023 •
edited by jira bot
Loading

Which version of NEXT UI Kit are you opening this bug for?

v5.x

Latest version

  • I have tested the latest version

No Hitachi confidential content

  • I'm not including any Hitachi confidential content

Current behavior 😯

When changing for a different page, the HvTooltip is still displayed because is attached to root, not to the component page.

Expected behavior 🤔

The component should have a container addressed to be attached on it and be unmounted when the container unmounts.

Steps to reproduce 🕹

Enter sandbox link:

And/or the steps to reproduce:

  1. Open a tooltip on page
  2. Go to a different page
  3. The tooltip is still displayed
Model.Server.-.Google.Chrome.2023-06-06.09-50-20.mp4

Context 🔦

Related to issue MLAAS-2269.

Your Environment 🌎

Device: [e.g. Laptop, Mobile, ....]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari, chrome]
Version [e.g. 22]
Other details [e.g. screen resolution 1920x1080, ...]
@stephaniegaspar stephaniegaspar added the ⌛ needs triage Waiting for a review. These issues haven't been looked at yet by a maintainer. label Jun 6, 2023
stephaniegaspar added a commit to stephaniegaspar/hv-uikit-react that referenced this issue Jun 6, 2023
@stephaniegaspar
fix(tooltip): fix container definition to append popper component. lu…
336ca33 
…mada-design#3411
HQFOX added a commit that referenced this issue Jun 6, 2023
@HQFOX
Merge pull request #3412 from stephaniegaspar/fix/3411
9a4cc44 
fix(tooltip): fix container definition to append popper component. #3411
@MEsteves22
Copy link
Contributor

MEsteves22 commented Jun 7, 2023 •
edited
Loading

Hello 👋

Thank you for your contribution. Your fix is on version @hitachivantara/[email protected].

@MEsteves22 MEsteves22 added 🐛 bug Something isn't working and removed ⌛ needs triage Waiting for a review. These issues haven't been looked at yet by a maintainer. labels Jun 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
🐛 bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@MEsteves22 @stephaniegaspar