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 wrong position causes tootlip flickering #14

Closed
YuraDubensky opened this issue Feb 22, 2018 · 4 comments
Closed

Tooltip wrong position causes tootlip flickering #14

YuraDubensky opened this issue Feb 22, 2018 · 4 comments

Comments

@YuraDubensky
Copy link

YuraDubensky commented Feb 22, 2018

Bug description:

When using tooltip-placement="left", tooltip tip occasionally is placed wrongly and overlaps toggle element element. So if mouse is placed close to the left border of toggle element - it causes tip to disappear immediately after show. This produces inifinite loop of show/hide tooltip while mouse is steady.

Root cause is that class 'placementClassPrefix-' is added to tip element after position is calculated.
Boostrap defines padding on ".bs-tooltip-left"so it affects tip element's width. And since tip position is calculated taking into account tip's width it result in tip being placed to much to the right and so overlapping toggle element.

Link to minimally-working plunker that reproduces the issue:

No need for a plunker - it is easily reproducible on demo page:
https://morgul.github.io/ui-bootstrap4/#!#tooltip

Steps to reproduce the issue:

Select tooltip placement as 'left' and place mouse close to the left edge of tooltip button.

Version of Angular, UIBS, and Bootstrap

Angular: 1.5.9

UIBS: 3.0.2

Bootstrap: 4.0.0

@YuraDubensky
Copy link
Author

YuraDubensky commented Feb 22, 2018

This is bug in action (note that right tooltip edge is overlapping with toogle element)
image

YuraDubensky added a commit to YuraDubensky/ui-bootstrap4 that referenced this issue Feb 22, 2018
Fix issue Morgul#14 - apply placement classes prior to tooltip placement - so calculated width is correct.
@YuraDubensky
Copy link
Author

@Morgul Can you please give a rough estimate on a date of next release?

@Morgul
Copy link
Owner

Morgul commented Mar 1, 2018

@YuraDubensky I'll do it as soon as #17 is merged in. I'm still waiting on some changes to that before it's good to go.

@Morgul
Copy link
Owner

Morgul commented Mar 13, 2018

Fixed by #18.

@Morgul Morgul closed this as completed Mar 13, 2018
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

2 participants