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

zIndex for placeholder span is set wrong #24

Open
lvostinar opened this issue Nov 28, 2013 · 2 comments
Open

zIndex for placeholder span is set wrong #24

lvostinar opened this issue Nov 28, 2013 · 2 comments
Labels

Comments

@lvostinar
Copy link

specifically this line:

if (isNaN(zIndex) || !zIndex) zIndex = 1;

is wrong, because default value of zIndex is auto which is default layer (same as zIndex 0)

so should be

if (isNaN(zIndex) || !zIndex) zIndex = 0;

@thisandagain
Copy link
Contributor

Awesome. Thanks for the catch. Would you mind sending along a PR?

@mcshaz
Copy link

mcshaz commented Aug 31, 2014

by setting z-ndex to 1 (i.e. this commit was removed - obviously had some downstream effects), the placeholder span will be given a value of zindex+1 (or 2). any jquery-ui dialogs, including datepickers will have a z-index of 1 if the z-index is not specifically set. the result of this is that the 'placeholder' span will sit on top of any dialog overlying the input. I can mock up in jsfiddle if you would like

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

No branches or pull requests

3 participants