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

fix problem for BS3 modal #665

Merged
merged 1 commit into from
Dec 15, 2014
Merged

fix problem for BS3 modal #665

merged 1 commit into from
Dec 15, 2014

Conversation

PEKTOP
Copy link
Contributor

@PEKTOP PEKTOP commented Oct 24, 2013

If you add input in modal datepicker show under modal shadow layer.
Set 'appendTo' for fix this problem, eg.:

$('.datepicker-input').datepicker({
    appendTo: "div.modal"
});

If you add input in modal datepicker show under modal shadow layer.
Set 'appendTo' for fix this problem, eg.:
<pre>
$('.datepicker-input').datepicker({
	appendTO: "div.modal"
});
</pre>
fadomire added a commit to fadomire/bootstrap-datepicker that referenced this pull request Nov 26, 2013
i guess that when calculating proper z-index for the datepicker, we should find the max z-index value of parents() and not stop at first

related to uxsolutions#533 uxsolutions#170 uxsolutions#683 uxsolutions#678 uxsolutions#656 uxsolutions#665 uxsolutions#634 uxsolutions#621 uxsolutions#609
@MaximilianoRicoTabo
Copy link

The @PEKTOP 's fix works Thanks dude.

@acrobat acrobat added this to the 1.3.1 milestone Dec 15, 2014
@acrobat
Copy link
Member

acrobat commented Dec 15, 2014

Thanks @PEKTOP!

acrobat added a commit that referenced this pull request Dec 15, 2014
fix problem for BS3 modal
@acrobat acrobat merged commit da85214 into uxsolutions:master Dec 15, 2014
This was referenced Dec 15, 2014
@acrobat acrobat removed this from the 1.3.1 milestone Dec 16, 2014
acrobat added a commit that referenced this pull request Dec 16, 2014
@acrobat
Copy link
Member

acrobat commented Dec 16, 2014

Reverted this PR in favor of #918. And moving this new option to the 1.4.0 release

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

Successfully merging this pull request may close these issues.

None yet

3 participants