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 with dimmer - working only once #48

Closed
alexmargineanu opened this issue Sep 20, 2013 · 5 comments
Closed

Modal with dimmer - working only once #48

alexmargineanu opened this issue Sep 20, 2013 · 5 comments

Comments

@alexmargineanu
Copy link

Hi,
I've used this lib for the first time, I love the design and feel!

I've found a couple of problems, this is related to the modal:

To show the modal I use:

$('.ui.modal.signoff').modal('toggle')

This is the inspected HTML:

<div class="ui dimmer page visible active">
<div class="ui signoff modal transition visible active" style="margin-top: -198px;">
...
</div>
</div>

After closing the modal, this is the HTML (no more classes on the modal div!):

<div class="ui dimmer page hidden">
<div class="" style="margin-top: -198px;">
....
</div>
</div>

Alex

@jlukic
Copy link
Member

jlukic commented Sep 20, 2013

Could you create a test case, i'm not able to replicate.

Here's a starting off point with semantic lib included
http://jsfiddle.net/Vhxfp/1/

@alexmargineanu
Copy link
Author

I've moved the html code to jsfiddle and the modal works, but without angular.js and the other js that I'm using. There's a conflict on my side, I will investigate and get back to you.
Thanks,
Alex.

@flessard
Copy link

I have the same problem here, and if i use jQuery 1.8.3, the class are removed, but is i use jQuery 2.0.3, the calss are not removed.

@bliuchak
Copy link

Yeap. Have the same problem on jquery 1.8.3
Here is an example.
http://jsfiddle.net/QgQJd/

Started from jquery 1.9.1 and higher - works fine.

Thanks

@jlukic
Copy link
Member

jlukic commented Sep 29, 2013

Tested and working with jQuery 1.8.3. Resolved?

@jlukic jlukic closed this as completed Sep 29, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants