Centering images in a div vertically

Question

I have this code for centering vertically the images in a bunch of divs.

function centerImages(parent, image) {
    var parent_height = $(image).parent().height(         


        

Answer 1

Try to this one,

div.clients li img { vertical-align:middle; }

Answer 2

Try this instead...

function centerImages(image) {
    var parent_height = $(image).parent().height();  
    var image_height = $(image).height();  
    var top_margin = (parent_height - image_height) / 2;  
    $(image).css( 'margin-top' , top_margin);
}
$(".clients li img").each(function() {
    centerImages(this);
});

You weren't actually passing in images, just the class selector. The above selects all relevant images and passes them in - there's no need for the parent parameter.

Answer 3

if your div have all the same height and contain only the image, that's a pure CSS solution:
http://jsfiddle.net/Tpy2w/

Relevant CSS

div {
  width: 300px;
  height : 300px; 
  line-height: 300px; 
  text-align: center;    
}

div img {
  vertical-align: middle;
}

Just set an height and the same line-height for the div. Then apply vertical-align: middle on the image