tonysamperi / ngx-popperjs

An Angular 8+ wrapper for @popperjs/core
https://tonysamperi.github.io/ngx-popperjs/
MIT License
65 stars 12 forks source link

BUG: v16 introduces small dot #54

Closed bersling closed 1 year ago

bersling commented 1 year ago

Describe the bug v16 introduces a small dot that I don't think is supposed to be there. The ngxp__container class seems to be responsible for it.

To Reproduce

Stackblitz: https://stackblitz.com/edit/stackblitz-starters-ntdnfh

Here's a v15 stackblitz, the dot isn't there https://stackblitz.com/edit/stackblitz-starters-yzdkhv

Expected behavior

No dot should appear.

Screenshots

Screenshot 2023-08-03 at 10 41 36

I've colored the box shadow of that dot in red so it can be seen better. The box shadow was already there however.

Desktop (if execution):

Versions

Additional Context

I also get a small white not fully closed popper-window in an app I have that uses ngx-popper since v16, however not in the stackblitz. I'll need to dig a bit deeper to reproduce that in the stackblitz though. I thought to give that context information here since (a) maybe it's related, (b) even if not, you might immediately know what's the problem.

Screenshot 2023-08-03 at 10 57 02

KikoCosmetics commented 1 year ago

Yeah it's caused by the workaound to the Chrome browser. I'll remove the border when hidden. In the meantime you can do

.ngxp__container.ngxp__hide {
    border: 0;
}

Cheers