Re: [pear-webmaster] Re: [PEAR-CVS] cvs: pearweb /public_html package-stats.php

On Sat, 2004-05-15 at 19:05, Martin Jansen wrote: > On Sat May 15, 2004 at 02:1544PM -0400, Daniel Convissor wrote: > > On Sat, May 15, 2004 at 07:41:39PM +0200, Martin Jansen wrote: > > > On Sat May 15, 2004 at 03:0524PM -0000, Helgi Þormar Þorbjörnsson wrote: > > > > +?> > > > > + <form action="package-stats.php" method="get"> > > > > + <table> > > > > + <tr> > > > > + <td> > > > > + <select name="cid" > > > > onchange="javascript:reloadMe();"> > > > > + <option>Select category ...</option> > > > > +<?php > > > > > > Please don't do this, as it makes editing the page really unhandy. > > > > Out of curiosity, what's "this." Flipping out of PHP mode? Or nesting > > with more than one space in HTML? Or??? > > Flipping out of PHP mode. > What's wrong with this ???? if there is any issue we can just do echo ''; on this ... Always found it less uglier to do it this way then just have it inside PHP, + this way we can get the HTML highlighted in editors ;) (not like it matters that much, but still) We maybe should think about switching over to some template engine, then we don't have html inside the php code and everything would be much easier :) My 2cents anyway. > -- > - Martin Martin Jansen > http://martinjansen.com/