Skip to content
This repository has been archived by the owner on Oct 22, 2024. It is now read-only.

Add popper.js as npm dependency #63

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

joachimschmidt557
Copy link
Member

Should I also add the initialization code to the HTML template?

@Xiphoseer
Copy link
Member

Can you look at djangocms-bootstrap4 and djangocms-cascade and try what they need? If the activate every popover individually, we don't need the global activation

@joachimschmidt557
Copy link
Member Author

@Xiphoseer Ok, I'll look into that.

@joachimschmidt557
Copy link
Member Author

I haven't found a way of having first-class support of popovers in djangocms-bootstrap4. I manually added the tags necessary for popovers via the advanced settings:

image

In order for the popover to actually show up, it was necessary to add the javascript code manually (in the template).

I'm not sure, but as far as I'm aware, djangocms-cascade also does not automatically include the javascript code for popovers, that would have to be done manually (e.g. in plugin templates if popovers are used in a plugin).

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

Successfully merging this pull request may close these issues.

2 participants