Height confused with width

Good patch. Thanks

Works for me should be committed.

Patched it manually and it works great now. Should be committed. Thanks so much.

Thanks for the patch suggestions. I found this exact problem in version 7.x-3.0-beta3. The changes were slightly different from the patch as this is Drupal 7...

Changed menu_icons.module:
line 307 as follows:

---   $css .= theme('menu_icons_css_item', array('mlid' => $item->mlid, 'path' => $image_url, 'size' => $info['width'], 'pos' => $pos));
+++         $css .= theme('menu_icons_css_item', array('mlid' => $item->mlid, 'path' => $image_url, 'size' => $info['width'], 'height' => $info['height'], 'pos' => $pos));

line 323 as follows:

--- 'variables' => array('mlid' => NULL, 'path' => NULL, 'size' => NULL, 'pos' => NULL),
+++   'variables' => array('mlid' => NULL, 'path' => NULL, 'size' => NULL, 'height' => NULL, 'pos' => NULL),

Changed menu_icons_css_item.tpl.php:
line 23 as follows:

---          height: <?php print $size?>px;
+++       height: <?php print $height?>px;

I re-added the image to the menu and the height of the items were now displaying properly. I hope this helps someone else in the future.

Updating version since this is a problem in the current version as well.

#5 works. thanks

Committed in http://drupalcode.org/project/menu_icons.git/commit/a9625bc, thanks!