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

Modal not showing after browser update #45

Open
nosizejosh opened this issue Jul 21, 2019 · 4 comments
Open

Modal not showing after browser update #45

nosizejosh opened this issue Jul 21, 2019 · 4 comments

Comments

@nosizejosh
Copy link

@PeppeL-G
I have been using this wonderful package with bootstrap 4 and it has worked fine until my chrome browser updated. Click events to show modals don't result in any errors neither does the modal show up.

Modal shows in older versions of firefox and Opera browsers.
I believe the issue may be related to this but not sure what to do make this package show the modals.

I need help fixing this issue, its critical!

@PeppeL-G
Copy link
Owner

I don't think the issue is related to that, because this package does not make use of that function.

Without seeing your code I don't know how to help you, but you say that you don't get any errors and that the modal doesn't show up. Can you check if the modal (the HTML code for it) is inserted into the DOM after you have called show()? It should be a direct child of <body>.

@PeppeL-G
Copy link
Owner

Also, this line of code could be the problem. I didn't spend much time on figuring out how it actually worked, but I thought that was a Meteor internal thing, so if it stops working it shouldn't work in any browser.

@PeppeL-G
Copy link
Owner

Or are you using Bootstrap 4? #44 mentions a workaround for getting this package to work in Bootstrap 4 (this package was only meant to be used for bootstrap 3).

@nosizejosh
Copy link
Author

nosizejosh commented Jul 23, 2019

Thank you so much. It turns out my browser had stored some data and cookies which was preventing normal functioning of modal.

Clearing those resolved the issue.

Thank you.

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

No branches or pull requests

2 participants