Allow to choose boundariesElement for actions' popover

[marcoambrosini]

Signed-off-by: Marco Ambrosini marcoambrosini@pm.me

[georgehrke]

@ma12-co Does this fix #1345 ? :)

[nickvergessen]

Does this fix #1345 ? :)

It allows you do fix it yeah, by specifying the element yourself.

[georgehrke]

Well, not really, since you can't set custom props for the actions component for secondary actions:
https://github.com/nextcloud/nextcloud-vue/blob/master/src/components/AppSidebar/AppSidebar.vue#L211

Let me see if adding .app-sidebar as a boundary works there.

[marcoambrosini]

Let me see if adding .app-sidebar as a boundary works there.

It does :)

[marcoambrosini]

But it actually should be an object, not a string :/

[marcoambrosini]

I'm testing now with the dom element body and it seems to work. Seems to be that we were passing in the string "body"

[marcoambrosini]

Can confirm: It works by passing the html body element, without the need of specifying a boundarieselement. I have a problem with the prop type check though. Should I just remove it?

EDIT: if I remove the prop type eslint will fail

[georgehrke]

@ma12-co See https://github.com/Akryum/v-tooltip/blob/fa672f6d6d2004d9cb285ce4551f12c9ca1f6540/src/components/Popover.vue#L114L117