Heine

  • home
  • drupal
  • drupal core commits
  • about
Home › Drupal Core Commits

Commit 261710 by dries

- Patch #520734 by Everett Zufelt, webernet | Dries, mgifford: theme_feed_icon() could use more meaningful alt text.

--- includes/theme.inc  2009/09/11 14:14:16     1.522
+++ includes/theme.inc  2009/09/11 15:17:00     1.523
@@ -1840,10 +1840,11 @@
  *   The url of the feed.
  * @param $title
  *   A descriptive title of the feed.
-  */
+ */
 function theme_feed_icon($url, $title) {
-  if ($image = theme('image', 'misc/feed.png', t('Subscribe to %feed-title', array('%feed-title' => $title)))) {
-    return '<a href="' . check_url($url) . '" class="feed-icon">' . $image . '</a>';
+  $text = t('Subscribe to @feed-title', array('@feed-title' => $title));
+  if ($image = theme('image', 'misc/feed.png', $text)) {
+    return '<a href="' . check_url($url) . '" title="' . $text . '" class="feed-icon">' . $image . '</a>';
   }
 }
 

No votes yet
  • Drupal Core
  • Download patch

Recent posts

  • Bugfix woes for Drupal 6
  • Arbitrary PHP code execution in modx 1.0.0 via CSRF
  • The #drupal consultant
  • Varnish vs. page cache graph
  • FAPI - How to decrease/increase the text limit of textfields?
more

Tags

Captcha CSRF Drupal Let's hope it doesn't become popular modx Performance PHP Pitfalls Planet Drupal Security V-PowerServer Varnish
more tags
  • home
  • drupal
  • drupal core commits
  • about

Copyright © 2009 by Heine Deelstra. All rights reserved.