Vanilla 1 is no longer supported or maintained. If you need a copy, you can get it here.
HackerOne users: Testing against this community violates our program's Terms of Service and will result in your bounty being denied.
Just jotting down some things I notice as I notice them with editing CSS.. <div class="CommentIcon" style="background-image:url('http://forums.hackwow.com/uploads/AccountPictures/4cd353e1b3bee4df59c3db62a3252b22.jpg');"> </div> Why isn't this an <img /> tag? Is it to make sure the image size stays exact? If so why bother with the ? <li><span>CommentAuthor</span><a href="http://forums.sourcepeek.com/account/1/">zeeg</a></li> <li><span>CommentTime</span>Aug 5th 2006 <em>edited</em></li></ul> Why is this a list? Why not a definition list if nothing else? HideComment('/ajax/switch.php', '1', '53', '225', 'Are you sure you wish to undelete this comment?', 'Are you sure you wish to delete this comment?', 'HideComment225'); return false;" This is repeated quite a bit, might be better to store the text inside the function, or in a variable. <fieldset> <legend>Add your comments</legend><form id="frmPostComment" method="post" action="http://forums.sourcepeek.com/post/"><input type="hidden" name="CommentID" value="0" /> Yell at me if I'm wrong, but I think <form> goes outside of <fieldset> :) <input type="submit" name="btnSave" value="Add your comments" class="Button SubmitButton AddCommentsButton" onclick="Wait(this, 'Wait');" /> </div> <a id="pgbottom" name="pgbottom"> </a> What's with the 's there? There are multiple (many) H1's on each page. In my humble opinion, and from my experiences, it is best to only use a single h1 describing exactly what that page is, then h2 for the major headers/keywords, h3 for bylines/subheadings, and that's usually as far as I go. Yes I'm nitpicking, but they're just basic ideas as I love how well Vanilla has kept up w/ standards :) Hope some of this helps!
This discussion has been closed.