From 592e0bc505c52a38952469bae0a081c224180bd8 Mon Sep 17 00:00:00 2001
From: Brion Vibber <brion@status.net>
Date: Tue, 9 Nov 2010 16:43:37 -0800
Subject: [PATCH] add title attribute on attachment list items

---
 lib/attachmentlist.php | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/lib/attachmentlist.php b/lib/attachmentlist.php
index f9ef7499e1..6e127af864 100644
--- a/lib/attachmentlist.php
+++ b/lib/attachmentlist.php
@@ -197,7 +197,10 @@ class AttachmentListItem extends Widget
     }
 
     function linkAttr() {
-        return array('class' => 'attachment', 'href' => $this->attachment->url, 'id' => 'attachment-' . $this->attachment->id);
+        return array('class' => 'attachment',
+                     'href' => $this->attachment->url,
+                     'id' => 'attachment-' . $this->attachment->id,
+                     'title' => $this->title());
     }
 
     function showLink() {
@@ -244,8 +247,9 @@ class AttachmentListItem extends Widget
                 case 'image/jpeg':
                     $thumb = (object)array();
                     $thumb->url = $enc->url;
-                    $thumb->width = 100;
-                    $thumb->height = 75; // @fixme
+                    // @fixme use the given width/height aspect
+                    $thumb->width = common_config('attachments', 'thumb_width');
+                    $thumb->height = common_config('attachments', 'thumb_height');
                     return $thumb;
             }
         }