Thanks for the feedback Dries. I'd be happy to take another crack
at this patch later this week. I have a business trip that will
keep me tied up until probably Friday at least.<br>
<br>
<br><br><div><span class="gmail_quote">On 3/6/06, <b class="gmail_sendername">Dries</b> <<a href="mailto:drupal-devel@drupal.org">drupal-devel@drupal.org</a>> wrote:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Issue status update for<br><a href="http://drupal.org/node/49575">http://drupal.org/node/49575</a><br>Post a follow up:<br><a href="http://drupal.org/project/comments/add/49575">http://drupal.org/project/comments/add/49575
</a><br><br> Project: Project<br> Version: cvs<br> Component: User interface<br> Category: bug reports<br> Priority: normal<br> Assigned to: Anonymous<br> Reported by: der<br> Updated by: Dries<br>
Status: patch (code needs review)<br><br>Code comment: we should wrap a check_url() around the screenshot URL<br>(depending on the format). We should add an alt-attribute for level 1<br>accessibility.<br><br><br>What happens with projects who configured a screenshot link? The
<br>Drupal project's screenshot link, for example, points to a gallery (see<br><a href="http://drupal.org/project/drupal">http://drupal.org/project/drupal</a>). Maybe we need to add a second URL so<br>we can do stuff like they do at the Firefox plugin directory (see
<br><a href="https://addons.mozilla.org/extensions/moreinfo.php?id=220&application=firefox">https://addons.mozilla.org/extensions/moreinfo.php?id=220&application=firefox</a><br>and note the 'more previews' link below the screenshot).
<br><br><br>Would you be able to work on this some more, so we can implement this<br>properly? I'd be happy to help test/review such functionality and roll<br>it out on <a href="http://drupal.org">drupal.org</a>.<br><br>
<br><br><br>Dries<br><br><br><br>Previous comments:<br>------------------------------------------------------------------------<br><br>Wed, 15 Feb 2006 21:09:41 +0000 : der<br><br>Per the related discussion [1] on the infrastructure mailing list, there
<br>needs to be a way to embed an image in the description of a project.<br>This is particularly useful for showing theme screenshots.<br>[1]<br><a href="http://lists.drupal.org/archives/infrastructure/2006-02/msg00154.html">
http://lists.drupal.org/archives/infrastructure/2006-02/msg00154.html</a><br><br><br><br><br>------------------------------------------------------------------------<br><br>Sat, 04 Mar 2006 05:35:47 +0000 : der<br><br>Attachment:
<a href="http://drupal.org/files/issues/project.module_1.patch">http://drupal.org/files/issues/project.module_1.patch</a> (1.69 KB)<br><br>Here's a patch that uses the existing screenshots db field for the img<br>tag source.
<br><br><br><br><br>------------------------------------------------------------------------<br><br>Mon, 06 Mar 2006 15:09:14 +0000 : der<br><br>Just a comment on this patch. The intent of this feature request was to<br>
allow screenshot images to be displayed for "theme" projects. But the<br>patch doesn't limit it to any particular project type. My assumption<br>is that it could be controlled in the stylesheet.<br><br><br><br>
<br></blockquote></div><br>