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

pTooltip: tooltip creates overflow of body #15599

Closed
Emiliorth opened this issue May 17, 2024 · 2 comments
Closed

pTooltip: tooltip creates overflow of body #15599

Emiliorth opened this issue May 17, 2024 · 2 comments
Labels
Status: Needs Triage Issue will be reviewed by Core Team and a relevant label will be added as soon as possible

Comments

@Emiliorth
Copy link

Describe the bug

In PrimeNG 17.16.1 when tooltip code is added at the end of , for a less than second it's code takes some height, making body have overflow if your site is set to have height of 100vh
In previous versions of PrimeNG, like 15xx there was no such a problem

Environment

Angular 17
PrimeNG 17.16.1
20240516_123415_chrome

Reproducer

No response

Angular version

17.3.0

PrimeNG version

17.16.1

Build / Runtime

Angular CLI App

Language

TypeScript

Node version (for AoT issues node --version)

21.6.0

Browser(s)

Chrome

Steps to reproduce the behavior

  1. Add pTooltip to anything
  2. Make div inside body to have height: 100vh
  3. After hovering on pTooltip element, p-tooltip will append to body, consuming height, so it will appear below our 100vh wrapper, giving body overflow

Expected behavior

Like in PrimeNG 15.x.x, appended p-tooltip to the body should not consume any height not for even 1ms so the overflow will not appear

@Emiliorth Emiliorth added the Status: Needs Triage Issue will be reviewed by Core Team and a relevant label will be added as soon as possible label May 17, 2024
@mehmetcetin01140
Copy link
Contributor

Hi, in the latest PrimeNG version (17.17.0), this issue has been fixed. Could you please try again with the updated version?

Example (v17.17.0) = https://stackblitz.com/edit/yqjz5u-c2d5dk?file=src%2Fapp%2Ftooltip-position-demo.html

I'm closing the issue.

@Emiliorth
Copy link
Author

Works flawlessly! Thank you for the information :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Needs Triage Issue will be reviewed by Core Team and a relevant label will be added as soon as possible
Projects
None yet
Development

No branches or pull requests

2 participants