So if you want to use an imagecache preset scaled to larger than variable_get('menu_icons_file_validate_image_resolution', '45x45'), you're going to get some pixelisation.

The attached patch will fix that, but of course if you've been using menu_icons for some time, then decide you want to use it with imagecache, you will have to re-upload all your images, so I think a more robust fix will be storing both the original and scaled image and defaulting to the scaled if an imagecache preset is not specified.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Matthew Davidson’s picture

Okay, attach the patch this time...

skilip’s picture

Status: Needs review » Reviewed & tested by the community

Thanks for the patch Matthew! It looks quite alright and I'll commit this asap.

skilip’s picture

Status: Reviewed & tested by the community » Fixed

Committed to HEAD. Thanks!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.