[pLog-svn] r5298 - plog/branches/lifetype-1.2/templates/summary

[Oscar Renalias]

Yes, I forgot about the "alt" attribute. I will add that later on or tomorrow.

But speaking of usability, there is no possibility to use an alt
attribute when using an image as a background to a div tag so I
figured it'd be better to be consistent with our usability rules
rather than respecting the ratio of a picture... :)

Oscar

On 4/11/07, BalearWeb <balearweb at balearweb.com> wrote:
> Hello,
>
> The reason why the image was displayed as a background was that I think
> it is much nicer to have a picture cut-off, than resized without
> respecting the original proportions of the picture, specially when it is
> a picture of a face.
>
> For example, if you resize a 120x60px face to 60x60 it can look awful. I
> contrast, if you cut-off and just show a part of the 120x60px, you might
> end up seeing part of the face, but at least you face will not be narrowed.
>
> Never mind, it is not very important. But if we are going to use the img
> element, then we need al "alt" text:
>
> - <img src="{$picture->getPreviewLink()}"  width="60" height="60" class="postgrav" />
>
> + <img src="{$picture->getPreviewLink()}" width="60" height="60"
> class="postgrav" alt="{$postOwner->getUsername()|strip_tags} />
>
> Kind regards,
>
> Elena
>
> oscar at devel.lifetype.net wrote:
> > -     <div style="border:solid #cccccc 1px;width:58px;height:58px;background:url({$picture->getPreviewLink()}) no-repeat center" class="postgrav">&nbsp;</div>
> > +          <img src="{$picture->getPreviewLink()}"  width="60" height="60" class="postgrav" />
> >
>
>
> _______________________________________________
> pLog-svn mailing list
> pLog-svn at devel.lifetype.net
> http://limedaley.com/mailman/listinfo/plog-svn
>