Open atwellpub opened 8 years ago
The popups and ACF repeater fields are probably added after dom loaded
You have to attach the handler like this:
jQuery("body").on('click', '.dynamic-element', function () {
});
Not sure how the library handles this.
Why did you choose: http://vadimsva.github.io/popModal/ ? The styles look not so great
We are handing it with the 'On', but it's still not working, which makes me think it might be an issue with popModal.
https://github.com/inboundnow/inbound-mailer/blob/master/classes/class.token-engine.php#L143-L158
I needed a simple and lightweight modal class that could be styled easily. How it is used I thinks looks decent : /
Yeah that looks okay. I was looking at the libraries demo.
I guess the main thing is when we are adding new libs to the code base that we don't duplicate functionality.
We have the old old old popover thing for other settings pages too. We should remove the old stuff when adding new stuff that does the same thing
On Sep 25, 2015, at 12:39 PM, Hudson Atwell notifications@github.com wrote:
We are handing it with the 'On', but it's still not working, which makes me think it might be an issue with popModal.
https://github.com/inboundnow/inbound-mailer/blob/master/classes/class.token-engine.php#L143-L158
I needed a simple and lightweight modal class that could be styled easily. How it is used I thinks looks decent : /
— Reply to this email directly or view it on GitHub.
PS there is a fix for that ACF warning: http://support.advancedcustomfields.com/forums/topic/wp_htmledit_pre-is-deprecated/
On Fri, Sep 25, 2015 at 12:39 PM, Hudson Atwell notifications@github.com wrote:
We are handing it with the 'On', but it's still not working, which makes me think it might be an issue with popModal.
https://github.com/inboundnow/inbound-mailer/blob/master/classes/class.token-engine.php#L143-L158
I needed a simple and lightweight modal class that could be styled easily. How it is used I thinks looks decent : /
— Reply to this email directly or view it on GitHub https://github.com/inboundnow/inbound-mailer/issues/57#issuecomment-143333617 .
Created issue here: https://github.com/vadimsva/popModal/issues/15