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 window affecting input focus #108

Closed
ivandotv opened this issue Jul 25, 2013 · 5 comments
Closed

Modal window affecting input focus #108

ivandotv opened this issue Jul 25, 2013 · 5 comments

Comments

@ivandotv
Copy link

I've seen that there have been a similar z-index issue regarding jquery ui dialog however I'm trying to use spectrum in combination with a very popular modal component:
Magnific Popup

And I've can't resolve the z-index issue in regards to that component.
I've setup a little test here (z-index on the popup is 1042)
http://jsfiddle.net/iki_xx/Vfm6X/3/

@bgrins
Copy link
Owner

bgrins commented Jul 26, 2013

These modal issues are always tricky. Do you have any ideas on how to fix it (it doesn't seem related to z-index this time)?

@ivandotv
Copy link
Author

Yeah it seems that it's not z-index related. I'm trying to figure out what's going on. It's very strange.

@dbeilharz
Copy link

I don't think this will work for you, but a quick hack I found when changing from jquery ui's dialog to bootstrap modal was to add
textInput.click(function() {$(this).focus(); });
on around line 279 of spectrum.js

I tried to implement this in your jsfiddle but with no success. It also doesn't seem to work to well in IE but that could just be my setup which has a few IE bugs.
This is just a quick hack that worked for me as I couldn't find the cause of the problem.
Thought I would just post this in case anyone else had trouble implementing with bootstrap modal.

@bgrins
Copy link
Owner

bgrins commented Sep 18, 2013

Thanks for sharing, it may be useful to @ikixxx trying to figure out his issue with the modal he is using.

@ivandotv
Copy link
Author

In the end, I've changed the design of my app to circumvent the issue.

@bgrins bgrins closed this as completed Sep 19, 2013
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

3 participants