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

IE7 & IE8: Text fields within a parent with opacity filter show no text #3552

Closed
wants to merge 1 commit into from
Closed

IE7 & IE8: Text fields within a parent with opacity filter show no text #3552

wants to merge 1 commit into from

Conversation

tomups
Copy link
Contributor

@tomups tomups commented May 21, 2012

Seems there is a bug in IE<9 where text fields (and textareas) that are in a container which has filter: alpha(opacity=[number]); applied are not re-rendered when being typed in.

Curiously enough, if you move your mouse out of the parent with the filter, the text will magically appear / update.

A straightforward use case where this happens is having a form inside a fade-in modal, as the .fade.in class has filter: alpha(opacity=100);. When you type, nothing will come up (but you can see the text cursor moving), and if you move the mouse into the overlay area, the text will appear/update.

Here is a test page:

http://www.cactilio.com/bootstrap-filter-test.html

A fix would be to not use a filter in IE<9 for .fade.in. It doesn't really give anything for IE<9 as the modals just pop-in and have no fading animation anyway!

@brettgoulder
Copy link

I agree, tested in XP IE7. https://github.com/raymccoy/bootstrap/commit/f9744ff52207617ec4610809603f34d7eba1d077 appears to have been a breaking change

@mdo
Copy link
Member

mdo commented May 29, 2012

Merged into 2.0.4-wip by 642f751. Thanks!

@mdo mdo closed this May 29, 2012
cvrebert added a commit to cvrebert/bs-css-hacks that referenced this pull request May 13, 2014
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

Successfully merging this pull request may close these issues.

None yet

3 participants