Currently, the Site Logo content type creates the <img> tag HTML, rather than using theme('image'). The latter allows themes to alter the image with hook_preprocess_image().

This is a simple change from:

$image = '<img src="' . $logo . '" alt="' . t('Home') . '" />';

to:

$image = theme('image', array('path' => $logo, 'alt' => t('Home')));
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

joelstein created an issue. See original summary.

joelstein’s picture

Status: Active » Needs review
FileSize
660 bytes
joelstein’s picture

Issue summary: View changes
carsonw’s picture

Status: Needs review » Reviewed & tested by the community

This is a super simple commit that works on many websites that we've patched. Let's get 'er included!

rivimey’s picture

Seems good to me. Proposed for 1.13

  • japerry committed d4441f0 on 7.x-1.x authored by joelstein
    Issue #2606518 by joelstein: Use theme_image() for site logo
    
japerry’s picture

Status: Reviewed & tested by the community » Fixed

+1 to less html in modules ;) Fixed!

Status: Fixed » Closed (fixed)

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