[development] Re: [bug] Allow embedded images in project description

David Reed dreed10 at gmail.com
Mon Mar 6 15:51:33 UTC 2006


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.



On 3/6/06, Dries <drupal-devel at drupal.org> wrote:
>
> Issue status update for
> http://drupal.org/node/49575
> Post a follow up:
> http://drupal.org/project/comments/add/49575
>
> Project:      Project
> Version:      cvs
> Component:    User interface
> Category:     bug reports
> Priority:     normal
> Assigned to:  Anonymous
> Reported by:  der
> Updated by:   Dries
> Status:       patch (code needs review)
>
> Code comment: we should wrap a check_url() around the screenshot URL
> (depending on the format).  We should add an alt-attribute for level 1
> accessibility.
>
>
> What happens with projects who configured a screenshot link?  The
> Drupal project's screenshot link, for example, points to a gallery (see
> http://drupal.org/project/drupal).  Maybe we need to add a second URL so
> we can do stuff like they do at the Firefox plugin directory (see
>
> https://addons.mozilla.org/extensions/moreinfo.php?id=220&application=firefox
> and note the 'more previews' link below the screenshot).
>
>
> Would you be able to work on this some more, so we can implement this
> properly?  I'd be happy to help test/review such functionality and roll
> it out on drupal.org.
>
>
>
>
> Dries
>
>
>
> Previous comments:
> ------------------------------------------------------------------------
>
> Wed, 15 Feb 2006 21:09:41 +0000 : der
>
> Per the related discussion [1] on the infrastructure mailing list, there
> needs to be a way to embed an image in the description of a project.
> This is particularly useful for showing theme screenshots.
> [1]
> http://lists.drupal.org/archives/infrastructure/2006-02/msg00154.html
>
>
>
>
> ------------------------------------------------------------------------
>
> Sat, 04 Mar 2006 05:35:47 +0000 : der
>
> Attachment: http://drupal.org/files/issues/project.module_1.patch (1.69KB)
>
> Here's a patch that uses the existing screenshots db field for the img
> tag source.
>
>
>
>
> ------------------------------------------------------------------------
>
> Mon, 06 Mar 2006 15:09:14 +0000 : der
>
> Just a comment on this patch.  The intent of this feature request was to
> allow screenshot images to be displayed for "theme" projects.  But the
> patch doesn't limit it to any particular project type.  My assumption
> is that it could be controlled in the stylesheet.
>
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.drupal.org/pipermail/development/attachments/20060306/300b381f/attachment.htm


More information about the development mailing list