Popover overflows bottom about bootstrap.native HOT 10 CLOSED

I have an idea on how to improve this, will let you know asap, perhaps tomorrow.

from bootstrap.native.

Works as intended. Many thanks for this enhancement!

from bootstrap.native.

@jcorporation I've played with the code a bit, I made it work to take this case into account and well... it's not right, mostly because the new positioning system based on position fixed for the tooltip and a much more simple algorithm makes Popovers always stick to the viewport edges, even if their trigger elements are not in the viewport at all. In short, it solves one niche case and produces an even bigger problem for everything else.

In my mind a solution can be to use a custom CSS class with a top: YOUR_VALUE !important position for your Popover and similar for it's arrow, since this is a very specific use case. It might also help us to implement the inserted.bs.popover event (it shouldn't be hard at all) to update your Popover position however you want when this event triggers. Even better we can also add updated.bs.popover but I think all this should benefit the community at large.

What do you think?

Happy New Year!!

from bootstrap.native.

This additional events should solve my problem. Could you add this events also to the dropdown component?

As reference my current patch to solve this: https://github.com/jcorporation/bootstrap.native/commit/73aa96402667ceb8e75fcc4166bf763d28d361a9#diff-73e8e7ae57dcf21616bd61f7eac8e8190db171f06e1f908cd7ae724b95b9c379

from bootstrap.native.

Let me finish updating the tests and I'll see what I can do. Meanwhile let me know what you think regarding:

• where you think the inserted event should trigger? I think an updated event would work better since it triggers on both inserted and updated, in fact it might be just to add to our update prototype method update () => { styleTip(this); /* trigger updated here */ }
• same question for Dropdown, similar context, however the dropdown menu is always in the DOM, so on updated would again work better.

What do you think?

from bootstrap.native.

We can use only an updated event. It should trigger after the position styles are applied and before the dom becomes visible.

from bootstrap.native.

@jcorporation I've implemented the feature, however I'm having problems with the Cypress testing suite. The same old fix one crap 2 more pop.

from bootstrap.native.

@thednp : notify me, as soon it can be tested

from bootstrap.native.

@jcorporation the implementation is live, since this morning. I still have some testing scripting to fix.

from bootstrap.native.

Great, I will test it.

from bootstrap.native.