The e-mail logo in the order confirmation message, should have alt and title added.

Comments

lsolesen’s picture

lsolesen commented
Status: Active » Needs review

Pushed to Gerrit.

bojanz’s picture

bojanz commented 
	-      $replacements[$original] = theme('image', array('path' => theme_get_setting('logo')));
	+      $replacements[$original] = theme('image', array('path' => theme_get_setting('logo'), 'item' =>
 array('alt' => variable_get('site_name'), 'title' => variable_get('site_name'))));

Aren't alt and title supposed to be on the same level as path? The item subarray would not be used?
http://api.drupal.org/api/drupal/includes!theme.inc/function/theme_image/7

lsolesen’s picture

lsolesen commented

Hm. I wonder what docs I was reading :/ Did a new push.

jsacksick’s picture

jsacksick commented
Status: Needs review » Fixed

Merged, thanks.

Status: Fixed » Closed (fixed)

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