-
-
Notifications
You must be signed in to change notification settings - Fork 78.8k
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
Arrow out of popover #23846
Comments
Can you make a CodePen of this issue please ? As it mentionned in our contributing guidelines |
@Johann-S I thought it's obvious, but here you go: https://jsbin.com/horagubehe. See that horizontal as well as vertical position is buggy. |
it seem's has a CSS problem because Popper.js position correctly our arrow but our arrow width is too high 🤔 yes @alecpl but that's easier for us to make a fix with a reproducible bug in live 👍 |
@Johann-S: Popper.js already takes care to include the arrow width in its computations (theorically), make sure the most outer arrow element has the right width set. edit: looks like the problem is what I pointed above. edit 2: If you get rid of the Note that the correct width is needed to position the arrow properly, and the margins are used to always ensure some spacing between the arrow and the popover's edge. |
Thank you so much @FezVrasta 👍 if you want to do that changes do not hesitate |
Maybe #23820 should be considered first. |
Can you check this https://codepen.io/anon/pen/gxNxZw ? I can make PR to fix this Edit: |
I didn't saw #23820 |
I was wrong. #23820 doesn't fix this issue. |
this CodePen : http://codepen.io/anon/pen/gxNxZw seems fine @wojtask9, isn't it ? |
@Johann-S |
the popover bottom left lost his left border ?? |
It seems floating-ui/floating-ui#489 fixe this issue, so when @FezVrasta will ship a new release of Popper.js we'll close this issue 👍 |
Released as v1.12.8 |
The new release of Popper.js fixe only one part of this issue see : https://codepen.io/Johann-S/pen/qVXEaX But IMO the first issue is on our CSS here (see the first Popover) |
yes, you still need to perform the changes I described in this comment |
Popover arrow can be placed incorrectly when button on the edge of the screen.
Tested with most recent Bootstrap4 build.
The text was updated successfully, but these errors were encountered: