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.

CommentFileSizeAuthor
#1 menu_icons.module-unwanted_resize.patch804 bytesMatthew Davidson

Comments

Matthew Davidson’s picture

Matthew Davidson commented
StatusFileSize
new menu_icons.module-unwanted_resize.patch804 bytes

Okay, attach the patch this time...

skilip’s picture

skilip commented
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

skilip commented
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.