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

Why use filter if you don’t support modern browsers anyway? #4

Closed
mathiasbynens opened this issue Aug 19, 2011 · 8 comments
Closed

Comments

@mathiasbynens
Copy link

From the gradients section:

filter: e(%("progid:DXImageTransform.Microsoft.gradient(startColorstr='%d', endColorstr='%d', GradientType=0)",@startColor,@endColor)); // IE6 & IE7

On the other hand, if you do want this to work in IE6 and IE7, you could just drop the -ms-filter declaration, as filter works in any IE6+.

@fat
Copy link
Member

fat commented Aug 22, 2011

We're using it for ie 7+ support now :)

@fat fat closed this as completed Aug 22, 2011
@mathiasbynens
Copy link
Author

Not so fast! :)

There’s still one occurence of -ms-filter that could/should be omitted:

-ms-filter: "progid:DXImageTransform.Microsoft.gradient(startColorstr='#00000000', endColorstr='#15000000')";
filter: "progid:DXImageTransform.Microsoft.gradient(startColorstr='#00000000', endColorstr='#15000000')";

Just the filter will do.

@fat
Copy link
Member

fat commented Aug 22, 2011

i believe -ms-filter is for ie8 http://www.quirksmode.org/css/opacity.html

@mathiasbynens
Copy link
Author

No it’s not. Try it.

@fat
Copy link
Member

fat commented Aug 23, 2011

"If you don’t use this order, IE8-as-IE7 doesn’t apply the opacity, although IE8 and a pure IE7 do."

@fat
Copy link
Member

fat commented Aug 23, 2011

Hm... that might just be with the opacity though... I'll check.

@fat fat reopened this Aug 23, 2011
@fat
Copy link
Member

fat commented Aug 23, 2011

8a27d28

thanks for hassling me on this ;D

@fat fat closed this as completed Aug 23, 2011
@mathiasbynens
Copy link
Author

If you don’t use this order, IE8-as-IE7 doesn’t apply the opacity, although IE8 and a pure IE7 do.

Don’t believe the hype! Always try it out for yourself.

thanks for hassling me on this ;D

No problem :)

@mdo mdo mentioned this issue May 4, 2013
@ghost ghost mentioned this issue Nov 4, 2013
muhammadrizuan pushed a commit to muhammadrizuan/bootstrap that referenced this issue Apr 8, 2014
Fixes issue found if user specifies font-size inherit in _options.scss. ...
DocX pushed a commit to DocX/bootstrap that referenced this issue Sep 16, 2014
Require sass-rails to avoid issues where bootstrap-sass is used in an engine
DocX pushed a commit to DocX/bootstrap that referenced this issue Sep 16, 2014
DocX pushed a commit to DocX/bootstrap that referenced this issue Sep 16, 2014
Updated handling of less escaped strings (Closes twbs#4)
glebm added a commit that referenced this issue May 29, 2015
mikejr83 pushed a commit to SumTotalSystems/bootstrap that referenced this issue Mar 29, 2016
Add license file
sarics314 pushed a commit to shore-gmbh/bootstrap that referenced this issue Aug 25, 2016
…-avatars to shore-bootstrap3

* commit '9b82d7a5cb79c574960acf88a46f17299a36f008':
  F [#MA-120] Update build and create new version
mdo pushed a commit that referenced this issue Oct 18, 2017
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

2 participants