-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Bootstrap 4 fix (for Popover style) #950
Comments
Thanks @benck . I just tried this, but it still doesn't want to play ball for me. I have Tether added (Before BS 4), and then when you click and an editable image, it comes up with: I've checked my CSS, and it all seems OK. If I remove the .popover-title, then it just shows an empty space (no content in it);
Any ideas? Cheers Andy |
The gist worked for me - I'm on Bootstrap 4 Alpha 6, and X-Editable wasn't working until I applied this. I am going to replace the pixel graphics with FontAwesome (the spinner and close GIFs), and the only other issue I see with this fix is the buttons in the popover are oddly sized (probably again slight code/CSS changes needed) |
The only thing I am noticing is despite the default popover position being |
Yeah, I did it ;) I made some improvements and now it's working perfectly |
Don't know if it's still relevant to this issue, but bootstrap 4 will move to Popper.js instead of Tether.js, so it might(will) affect this. |
Yes, it's broken now that Bootstrap 4 Beta 1 uses Popper.js |
The fix for Bootstrap 4 Beta is to change innerCss property from '.popover-content' to '.popover-body'' (line 4712). Also, setPostion should contain an empty function, like this: setPosition: function () { |
Hi, mostly it worked for me, but somehow it conflicts with this: |
hi, can someone give a hint of the css? (its indeed poping far right from the element) |
Works fine, but remote source still doesn't work |
For those of you who are struggling to implement it on Bootstrap 4, maybe this can help. For Bootstrap 4 its work, check this #1189 (comment) |
I just created a bootstrap 4 fix for Popover style (Inline style works no problem) which includes the latest bootstrap 4.0.0 alpha 5 and jquery 3.1.1.
The main change is to comment out the "setPosition" part since the position is handled by Tether from BS4.
Make sure you include Tether because it's required by Tooltip in BS4 and Fontawesome is used to replaced the old removed glyphicon buttons. (make sure to include Fontawesome if you don't want to change the code)
https://gist.github.com/benck/ed955584762b5667e8c6c270119c1fd3
The text was updated successfully, but these errors were encountered: