Skip to content

[BUG] The tooltip remains displayed when deleting an element with WrapperTooltip #924

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

Closed
finaly55 opened this issue Jan 30, 2023 · 1 comment · Fixed by #926
Closed

[BUG] The tooltip remains displayed when deleting an element with WrapperTooltip #924

finaly55 opened this issue Jan 30, 2023 · 1 comment · Fixed by #926
Labels

Comments

@finaly55
Copy link

finaly55 commented Jan 30, 2023

Describe the bug
Scenario, I have a delete button on an element, the button is enclosed by a WrapperTooltip.
I hover the button, tooltip shows.
I click the button, button goes away, tooltip remains.

Version of Package
v5.7.0

To Reproduce

<TooltipProvider>
    <Tooltip id={"informative"}>
    </Tooltip>
    <main>
        <TooltipWrapper tooltipId={'informative'} content={'textContentTooltip1'}>
             <btn onClick="Close()">close</btn>
        </TooltipWrapper>
        <TooltipWrapper tooltipId={'informative'} content={'textContentTooltip2'}>
             <btn onClick="Close()">close</btn>
        </TooltipWrapper>
        <TooltipWrapper tooltipId={'informative'} content={'textContentTooltip3'}>
             <btn onClick="Close()">close</btn>
         </TooltipWrapper>
    </main>
</TooltipProvider>

Thank you in advance for your answers ! :)

@gabrieljablonski
Copy link
Member

gabrieljablonski commented Jan 30, 2023

Thanks for reporting, v5.7.2 has been released with a fix for this. Please let us know if it works correctly.

It should be noted that the new code watches for the anchor element being removed from the DOM, so you should make sure the wrapper also gets removed when the button does.

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

Successfully merging a pull request may close this issue.

2 participants