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

Carousel Responsive Issue In Chrome #21966

Closed
benjaminthedev opened this issue Feb 11, 2017 · 7 comments
Closed

Carousel Responsive Issue In Chrome #21966

benjaminthedev opened this issue Feb 11, 2017 · 7 comments

Comments

@benjaminthedev
Copy link

Hello there,

I have created a Carousel and when responsive in Chrome it squashes the image rather than scale down. Does anyone else have this issue? This is happening on all my desktop devices in Chrome.

Here is an image to highlight it.

http://imgur.com/a/5lOr8

This is also happening on BS4 doc Carousel page:

https://v4-alpha.getbootstrap.com/components/carousel/#options

Cheers

@josmaissan
Copy link
Contributor

Same as #21611 I think.

@benjaminthedev
Copy link
Author

Yes thanks it is and you have helped me. Thanks so much. This should be fixed tho right??

@josmaissan
Copy link
Contributor

josmaissan commented Feb 12, 2017

This should be fixed tho right??

I hope they fix. In my opinion they should, but it's not added to milestone v4.0.0-beta yet...
I have just added an example on JSFiddle too, in case they want that.

@benjaminthedev
Copy link
Author

Wicked, yeah they should at some point do that. What is the link to the fiddle if you do not mind me asking? Cheers.

@josmaissan
Copy link
Contributor

It's in my last post at #21611 ofcourse, but is: https://jsfiddle.net/josmaissan/5jsonocq/15/

@kwhdever
Copy link

24 Feb 17
Confirm the carousel "image squashing" is still an issue, on Chrome Version 56.0.2924.87 (Windows 7)
Carousel worked fine in A5.

To demo:

  1. Go to URL:
    https://v4-alpha.getbootstrap.com/components/carousel/
  2. Narrow the viewport
  3. Observe the image crush. Images do not resize.

@josmaissan
Copy link
Contributor

@kwhdever Your right! #21611 is still open. 😉

@mdo mdo added the has-pr label Apr 8, 2017
@mdo mdo added this to the v4.0.0-beta milestone Apr 8, 2017
@mdo mdo mentioned this issue Apr 8, 2017
@mdo mdo closed this as completed in #22392 Apr 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants