-
Notifications
You must be signed in to change notification settings - Fork 211
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
Not working in IE8 and IE9+ #169
Comments
Thanks for reporting, @Komald. We will look into this issue asap. |
Just saw that this demo is using v1.0.1. Can you please check with 1.2.0 https://github.com/drublic/css-modal/blob/master/build/modal.css ? |
I tried by replacing the css and js with v1.2 but that wont work. Instead it disturbed the design completely. I don't know if I did this correct, but you can have a look |
That's just a problem with the wrong class: modals need .modal--show http://jsfiddle.net/seyfso6L/2/ |
Ok, I checked it, but still its not working as expected. If I click on Forgot Password then it should close the existing Sign In modal and open the Forgot Password modal. just I said the initial fiddle works for all browsers except IE. is it possible to do something on that script and start working on IE too? |
So I've checked this again. Currently the second modal opens "stacked" above the first one. This happens in IE9 and Chrome for me. Edit: Stackable modals don't seem to work according to my tests at the moment. I'll take care of this. |
Stackable modals should be fixed now. |
Closing this now. Please let me know if you have any problems with it. |
Hello There,
I am facing a situation for IE browser all versions. I am having a link to open a modal and another link on that modal to close current modal and open another one. This works perfectly well in rest of the leading browsers, but fails in IE. The processes executes well till opening first modal, but when I click link to open another modal, the first one closes (as expected) but does not opens the second.
Here is the fiddle link
http://jsfiddle.net/seyfso6L/
Hope this will help to understand the issue. Please have a look.
Thanks,
-Komal
The text was updated successfully, but these errors were encountered: