Remove null title attribute from service link images

The markup for the first link is:

<a href="http://blogs.icerocket.com/search?q=http%3A%2F%2Fwww.fixtheweb.net%2Fhome" title="Search IceRocket for links to this post." id="service-links-icerocket-1" class="service-links-icerocket" rel="nofollow"><img src="/sites/all/modules/service_links/images/icerocket.png" alt="Icerocket" title="" width="16" height="16" /></a>

Not sure why Window Eyes / IE8 is messing this up. The only thing that you might try is removing the empty title attribute from the image. A screen-reader should use the image alt, or if not available, and if no anchor text is available, the title attribute of the anchor.

Seeing as I also have the NVDA screen reader installed, I thought i'd see how it read the same snapshot. If I used the Tab key to move through the links everything was fine, but when I used the small arrow keys to move down the page, all the links were in 1 long line which was very confusing.

That is expected behavior with NVDA, it reads the entire line.

I'm glad this worked for you. That being said, I can't think of a reason that Window Eyes shouldn't work properly with title="", this, IMO, is a bug in Window Eyes.

Nevertheless, it would likely be good to remove title="" from the markup. Setting back to Active since the issue isn't 'fixed' in the module.

Updating title

This should be fixed into the core (http://api.drupal.org/api/function/theme_image) or into your theme rewriting the phptemplate_image() function, otherwise title attribute will be always added.

This appears to be fixed in theme_image() in Drupal 7, http://api.drupal.org/api/function/theme_image/7

function phptemplate_image($path, $alt = '', $title = '', $attributes = NULL, $getsize = TRUE) {
  if (!$getsize || (is_file($path) && (list($width, $height, $type, $image_attributes) = @getimagesize($path)))) {
    if (!empty($title)) {
      $attributes['title'] = $title;
    }
    $attributes = drupal_attributes($attributes);
    $url = (url($path) == $path) ? $path : (base_path() . $path);
    return '<img src="'. check_url($url) .'" alt="'. check_plain($alt) . '" ' . (isset($image_attributes) ? $image_attributes : '') . $attributes .' />';
  }
}

This should work, if you put it in your template.php (if doesnt work try also to rename the function like: theme_name_image)

many thanks crow, works!