5 replies to this topic

[Dvmby]

Hi, I'm Dvmby. My website is DvmbledoresArmy.com . I'd love to see what you have to say about it

Edited by Dvmby, 20 August 2007 - 03:40 PM.

[Olly]

Not a bad forum! I like it

[pete]

I liked the site the forum self well very basic really but keep work up any way.

[Pritthish]

hmm, isnt your real name Zach? if yes, i think i know you from a long time back
you used to be HPB's affiliate back when I was starting out.

Glad to see you got your own domain now though   

Anyway, i'd like to point some minor things out from a semantic coder's point of view
1) try using less containers..
ex:

<div id="lql">
		 <ul>
	 
		 <li><a href="http://dum-arm.com/index.php">DA News</a></li>
		 <li><a href="http://dum-arm.com/index.php?subaction=showfull&id=1185217821&archive=&start_from=&ucat=2&">Post-Harry Potter</a></li>
		 <li><a href="http://dum-arm.com/site/autonews.php">Automatic News</a></li>
		 <li><a href="http://dum-arm.com/mypotter">mypotter</a></li>
 
				 <li><a href="http://dum-arm.com/site/forums">Hog's Head Forums</a></li>
		 <li><a href="http://dum-arm.com/site/dvmbcast.php">Dvmbcast</a></li>
		 <li><a href="http://dum-arm.com/site/wiki">DvmbWiki</a></li>
		 <li><a href="http://hpspg.dum-arm.com/">HPSPG</a></li>
		 </ul>
		 <hr />
	 </div>

see here, you dont actually need the div...  you could directly apply id="lql" to the <ul> element. It may not seem much, but it adds up. I managed to slash over 6kb from my site's template just by removing unnecesary tags...

2) Now this is something which does not affect the visitors, but creates problems with search engines and sites like technorati.

<h3>August 24, 2007 @ 04:45AM</h3><h4>Harry Potter a Magical Read for Kids</h4>

the date doesnt actually need to be in a heading tag because, frankly, it's not a heading...  its just meta data which you should put in a <span> at the end of each post and use css to bring to the top.

and try not to make your news headlines as H4, try keeping them at H2..
the
<h2>Home/News</h2>

isnt necessary...   just get on to the news straight away

3) Small coding error

<ul id="SiteL">
<big>Site</big>
	<li><a href="http://dum-arm.com/index.php"><span title="You are here">»</span>Home/News</a></li>

you gotta put it as

<ul id="SiteL">
<li><big>Site</big></li>
	<li><a href="http://dum-arm.com/index.php"><span title="You are here">»</span>Home/News</a></li>

its a simple mistake, but do try to correct it

But i do love the visuals in the layout

Cheers!

[Weasel King]

Rich, give that kid pritthish a job.

[Pritthish]

Weasel King, on Dec 10 2007, 11:55 AM, said:

Rich, give that kid pritthish a job.

lol