Tickets :: [#3808] Thumbnails displayed instead of linked

6.0.0-beta1

7/6/25

Summary	Thumbnails displayed instead of linked
Queue	IMP
Queue Version	HEAD
Type	Enhancement
State	Resolved
Priority	1. Low
Owners
Requester	hackeron (at) gmail (dot) com
Created	04/21/2006 (7016 days ago)
Due
Updated	05/29/2006 (6978 days ago)
Assigned
Resolved	05/28/2006 (6979 days ago)
Milestone
Patch	No

05/29/2006 11:57:42 AM	ekg2002 (at) columbia (dot) edu	Comment #6	Reply to this comment
Content-disposition headers are in fact prioritized. I think the "click here to see a thumbnail" is fairly useless; you might as well click to the the larger image initially. I'd bet Chuck's comment about not making it configurable was made because this would make one more configuration option for the end-administrator and one more variable to store for a not-so-essential feature.

05/29/2006 08:28:07 AM	uhlar (at) fantomas (dot) sk	Comment #5	Reply to this comment
I think that no thumbnails should be displayed unless users configures it so. IMHO it should be decided according to Content-Disposition header and then to user's setup.

05/28/2006 04:09:34 AM	Chuck Hagenbuch	Comment #4 State ⇒ Resolved	Reply to this comment
Looks fine to me as is; I'm not going to make it configurable if it really doesn't need to be. Committed - thanks!

05/28/2006 01:28:47 AM	ekg2002 (at) columbia (dot) edu	Comment #3 New Attachment: thumbnail_screenshot.png	Reply to this comment
And now attached is a screenshot.

05/28/2006 01:28:07 AM	ekg2002 (at) columbia (dot) edu	Comment #2 New Attachment: thumbnail.diff	Reply to this comment
Attached is a two-line patch to make the thumbnail display inline. The only objection that I may have to it is that there is no option to turn it off / disable it for installs without proper resources. Let me know what you think is appropriate and I'll add it.

04/23/2006 11:18:49 PM	Jan Schneider	State ⇒ Accepted

04/21/2006 09:50:08 AM	hackeron (at) gmail (dot) com	Comment #1 Priority ⇒ 1. Low Type ⇒ Enhancement Summary ⇒ Thumbnails displayed instead of linked Queue ⇒ IMP State ⇒ New	Reply to this comment
Hey there, Currently, messages with images have links in the body: "View Thumbnail A large image named 19.04.06 046.jpg is attached to this message. Click HERE to view a thumbnail of this image." I really dislike that as if I have to click on every image individually to see a tiny thumbnail, I may as well just click on the full size images. Would be great if there was an option to show all thumbnails straight away in the body (performance hog, but very useful) and the link would be to show the full size.