patch to make carl_contest css and xhtml validate

Skinning and designing Serendipity (CSS, HTML, Smarty)
Post Reply
jkv
Regular
Posts: 6
Joined: Tue Nov 14, 2006 6:39 pm
Contact:

patch to make carl_contest css and xhtml validate

Post by jkv »

Problem:
- Use of comments inside comments(not xml valid)
- a </div> was in surplus

Update:
Finally got it sorted, if anyone uses this template please test the patch and reply if it break something...


unified diff:
http://www.unixcluster.dk/stuff/patches ... test.patch
Last edited by jkv on Fri Jan 12, 2007 5:53 pm, edited 1 time in total.
carl_galloway
Regular
Posts: 1331
Joined: Sun Dec 04, 2005 5:43 pm
Location: Andalucia, Spain
Contact:

Post by carl_galloway »

Hi jkv, I got your email, just been too busy to reply, and then saw this thread so forgive me for replying here.

Thanks for the heads up, I'll take a look at your suggestions over the next few days and email Garvin once I've been able to check them against my test server.
jkv
Regular
Posts: 6
Joined: Tue Nov 14, 2006 6:39 pm
Contact:

Post by jkv »

carl_galloway wrote:Hi jkv, I got your email, just been too busy to reply, and then saw this thread so forgive me for replying here.

Thanks for the heads up, I'll take a look at your suggestions over the next few days and email Garvin once I've been able to check them against my test server.
No problem...

Got the last problem sorted out, another </div> was misplaced, need to be placed before line 175 in entries.tpl, updated the patch to reflect this...

Speaking of debugging, do you have any 'need to know' hints on this subject?

At the moment im using a perl s///g to insert debug statements into the template im debugging. I want to know when a control statement starts and ends...
I wrote a onliner to help me debug, but it would really help with a GUI editor which can expand/shrink control statements...

perl -ple 's/{\/if}/<!-- \/if $. --> {\/if}/g ; s/({if.+?})/\1<!-- if $. -->/g ; s/({foreach.+})/\1<!-- foreach $. -->/g ; s/{\/foreach}/<!-- \/foreach $. -->{\/foreach}/g' < entries.tpl.orig > entries.tpl
Post Reply