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 IE7 z-index issue with modal #829

Closed
wants to merge 2 commits into from
Closed

Fix IE7 z-index issue with modal #829

wants to merge 2 commits into from

Conversation

fdaugan
Copy link
Contributor

@fdaugan fdaugan commented Feb 18, 2014

When a datepicker in within a BS modal, the dropdown is not visible in IE7 since IE7 return '0' and not 'auto' value for undefined attribute.
Credits : http://stackoverflow.com/questions/13074624/how-do-i-get-twitter-bootstrap-datepicker-to-work-in-ie7

Updated PR of #683

When a datepicker in within a BS modal, the dropdown is not visible in IE7 since IE7 return '0' and not 'auto' value for undefined attribute.
Credits : http://stackoverflow.com/questions/13074624/how-do-i-get-twitter-bootstrap-datepicker-to-work-in-ie7

Updated PR of #683
@eternicode eternicode added the IE label Mar 2, 2014
@eternicode eternicode added this to the 1.3.1 milestone Mar 2, 2014
@eternicode
Copy link
Contributor

Needs jshint fixing.

@robmcguinness
Copy link

+1

@jmshal
Copy link

jmshal commented Apr 8, 2014

I'm sure I'm missing something here, but didn't Bootstrap drop support for IE7? It states on the documentation site that they support Internet Explorer 8-11 on Windows.

@robmcguinness
Copy link

It's not just an IE7 issue, the plugin incorrectly calculates the z-index when inside a modal.

@jmshal
Copy link

jmshal commented Apr 8, 2014

Ah right, I understand, thanks for clearing that up.

+1

@acrobat acrobat modified the milestones: 1.3.2, 1.4.0 Dec 15, 2014
@acrobat
Copy link
Member

acrobat commented Dec 18, 2014

Fixed in #710

@acrobat acrobat closed this Dec 18, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants