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

Bootstrap 4 fix (for Popover style) #950

Open
benck opened this issue Nov 25, 2016 · 11 comments
Open

Bootstrap 4 fix (for Popover style) #950

benck opened this issue Nov 25, 2016 · 11 comments

Comments

@benck
Copy link

benck commented Nov 25, 2016

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

@benck benck mentioned this issue Nov 25, 2016
@youradds
Copy link

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:

image

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);

<div class="popover fade bs-tether-element bs-tether-enabled bs-tether-element-attached-middle bs-tether-element-attached-left bs-tether-target-attached-middle bs-tether-target-attached-right in" role="tooltip" id="popover544699" style="top: 0px; left: 0px; position: absolute; transform: translateX(1144px) translateY(438px) translateZ(0px);"><h3 class="popover-title">Enter a description</h3><div class="popover-content"> </div></div>

Any ideas?

Cheers

Andy

@Joyrex
Copy link

Joyrex commented Jan 26, 2017

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)

@Joyrex
Copy link

Joyrex commented Jan 26, 2017

The only thing I am noticing is despite the default popover position being top, it defaults to right - when declaring it via x-editable's placement data- attribute, it does render to top. However, the positioning is off - it wants to attach itself to the bottom of the element invoking the x-editable popover.

@EdnaldoNeimeg
Copy link

EdnaldoNeimeg commented Apr 20, 2017

Yeah, I did it ;)

I made some improvements and now it's working perfectly
(you'll might need to change your CSS)
https://gist.github.com/EdnaldoNeimeg/c7279ccade22d590ad89f4e0fe456742

image

@lvmajor
Copy link

lvmajor commented Aug 2, 2017

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.

@Joyrex
Copy link

Joyrex commented Aug 16, 2017

Yes, it's broken now that Bootstrap 4 Beta 1 uses Popper.js

@rustemas
Copy link

rustemas commented Sep 7, 2017

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 () {
(function() {}).call(this.container());
}

@ghost
Copy link

ghost commented Oct 14, 2017

Hi, mostly it worked for me, but somehow it conflicts with this:

https://github.com/thiagogmiranda/xeditable.ckeditor

@ghost
Copy link

ghost commented Oct 17, 2017

hi, can someone give a hint of the css? (its indeed poping far right from the element)

@Kopyov
Copy link

Kopyov commented Oct 18, 2017

Works fine, but remote source still doesn't work

@ilramdhan
Copy link

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)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

9 participants