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

add docs about scale .img-responsive svg for IE 11 #16782

Closed
alexander-akait opened this issue Jul 8, 2015 · 11 comments
Closed

add docs about scale .img-responsive svg for IE 11 #16782

alexander-akait opened this issue Jul 8, 2015 · 11 comments
Labels

Comments

@alexander-akait
Copy link

@media screen and (-ms-high-contrast: active), (-ms-high-contrast: none) {
    width: 100%;
}

Before hack:
ie11 - win8 1
After hack:
ie11 - win8 1 - 2

@cvrebert cvrebert added the css label Jul 8, 2015
@cvrebert
Copy link
Collaborator

cvrebert commented Jul 8, 2015

Please post a JS Bin demonstrating the problem and proposed fix.
And feel free to open a pull request for your proposed fix.

@alexander-akait
Copy link
Author

@cvrebert
IE version: 11.0.9600.17842
Update Version: 11.0.20 (KB3058515)
Without hack: http://output.jsbin.com/vurutizura/
With hack: http://jsbin.com/gulorudeyu/

@Johann-S
Copy link
Member

Confirmed on :
Win7 IE version: 11.0.9600.17843
Update Version: 11.0.20 (KB3058515)

@cvrebert
Copy link
Collaborator

X-Ref: #10073, #13996

@cvrebert
Copy link
Collaborator

@evilebottnawi That hack isn't feasible. It will cause non-SVG images to be stretched beyond their natural dimensions, which is undesirable; see #13996.

@alexander-akait
Copy link
Author

@cvrebert how about use hack for class .img-responsive with filename end on .svg?

@alexander-akait
Copy link
Author

@cvrebert hack with

.img-responsive {
    width: 100% \9;
}

Not working in ie11 and no docs about this bugs

@cvrebert
Copy link
Collaborator

how about use hack for class .img-responsive with filename end on .svg?

We ruled that out in the previous issues.

@alexander-akait
Copy link
Author

@cvrebert

.img-responsive {
    width: 100% \9;

    @media screen and (-ms-high-contrast: active), (-ms-high-contrast: none) {
        &[src$="svg"],
        .thumbnail > img[src$="svg"],
        .thumbnail a > img[src$="svg"],
        .carousel-inner > .item > img[src$="svg"],
        .carousel-inner > .item > a > img[src$="svg"] {
            width: 100%;
        }
    }
}

How about this construction, working in IE8-IE10, and IE11, http://dev.l-c-n.com/CSS3-selectors/browser-support.php, may be working in IE7, need testing

@cvrebert
Copy link
Collaborator

@evilebottnawi Like I said, we've already ruled out [src$="svg"] as insufficiently general; see #13996 (comment)

@mdo
Copy link
Member

mdo commented Aug 5, 2015

Punting with no reply. Happy to revisit though with a viable fix.

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

4 participants