Re: [SMARTY-DEV] Re: Bugreport: html_image shouldn't use border="0"

From: Date: Wed, 25 Feb 2004 16:19:15 +0000
Subject: Re: [SMARTY-DEV] Re: Bugreport: html_image shouldn't use border="0"
References: 1 2 3  Groups: php.smarty.dev 
Request: Send a blank email to php-smarty-dev+get-2147@lists.php.net to get a copy of this message
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. I think that adding ne {xhtml_image} is unecesary and confusign, because later we may find such XHTML incompliances with other tags. Regards, Muhammed Mamedov turkmenweb.com ----- Original Message ----- From: "messju mohr" <messju@lammfellpuschen.de> To: "Rainer Müller" <rainer.mueller@codingfarm.de> Cc: <smarty-dev@lists.php.net> 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 > > >

« previous php.smarty.dev (#2147) next »