Re: Re: Bugreport: html_image shouldn't use border="0"

On Wed, Feb 25, 2004 at 06:19:15PM +0200, Muhammed Mamedov wrote:
> I think that providing extra parameter is nice, if no border parameter is
> passed then border will be omitted, but when border="value" is passed it
> will be set.

That not backwards compatible. Currently if the border-attribute is
omitted it defaults to '... border="0" ...'.
 
> I think that adding ne {xhtml_image} is unecesary and confusign, because
> later we may find such XHTML incompliances with other tags.

And what do we do then? Making them xhtml-compliant and eventually
breaking html-compliance with that?

> Regards,
> Muhammed Mamedov
> turkmenweb.com
> 
> 
> 
> ----- Original Message -----
> From: "messju mohr" <[email protected]>
> To: "Rainer Müller" <[email protected]>
> Cc: <[email protected]>
> Sent: Wednesday, February 25, 2004 5:40 PM
> Subject: Re: [SMARTY-DEV] Re: Bugreport: html_image shouldn't use border="0"
> 
> 
> > On Wed, Feb 25, 2004 at 01:47:14AM +0100, Rainer Müller wrote:
> > > Rainer müller schrieb:
> > > >Hi,
> > > >currently html_image creates an img tag with the attribute border="0".
> This
> > > >isn't good because in XHTML there is no attribute border for the img
> tag.
> > > >border should not be used by default and should only be enabled with an
> > > >extra param border.
> >
> > Yes, great. That would break it for everybody who relied on border
> > defaulting to "0".
> >
> >
> > My suggestion for another solution involves two things:
> >
> > - Make html_image handle border=null (that is the null constant for an
> > unset value as in php) for an undefined border.
> >
> > {html_image} will still render border="0"
> > but
> > {html_image border=null} will omit the border
> >
> >
> > - For convinience we can add a new plugin {xhtml_image} that loads and
> > calls {html_image} with all parameters but uses null as default for
> > border instead of 0.
> >
> >
> > > >Raim
> > > Does nodoby noticed that? Or is there nobody interested in valid html?
> > >
> > > Raim
> >
> > --
> > Smarty Development Mailing List (http://smarty.php.net/)
> > To unsubscribe, visit: http://www.php.net/unsub.php
> >
> >
> >
> 
> -- 
> Smarty Development Mailing List (http://smarty.php.net/)
> To unsubscribe, visit: http://www.php.net/unsub.php

-- 
Smarty Development Mailing List (http://smarty.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php