Hi,

I think this ...

img, object, embed {
  max-width: 100%;
}

... should be adjusted to ...

img, object, embed {
  max-width: 100%;
  height: auto;
}

... respect aspect ratio.

Comments

migueltrindade’s picture

migueltrindade commented
Status: Active » Closed (works as designed)

There is not need to set "height: auto" since the height will be automatically calculated when the page was resized.

Wolfgang Reszel’s picture

Wolfgang Reszel commented

Well it's not needed if the IMG tags have no height. But Drupal normally renders IMG tags with width and height (which is a good decision).

Just have a look to this simple example:
http://jsfiddle.net/c2BXC/

migueltrindade’s picture

migueltrindade commented

Sorry Wolfgang Reszel, you are completely right. I just made a mistake in my tests :(
Fixed on dev release.

Wolfgang Reszel’s picture

Wolfgang Reszel commented
Status: Closed (works as designed) » Needs review

Sorry, I haven't tested the damn IE8/7. It also needs adding width: auto;

alexweber’s picture

alexweber commented
Status: Needs review » Fixed

Fixed in 2634cb2

alexweber’s picture

alexweber commented
Status: Fixed » Closed (fixed)