Hej,
Od strony HTML całkiem nieźle, aczkolwiek:
Kod
<div id="banner_main">www.olddomains.net - <strong>Expired Domains Names</strong><br />
Tutaj zamiast <br /> można dać w CSS display:block; dla stronga, a linia się przełamie.
Dalej zamiast <br /> akapity moim zdaniem.
Widziałbym też zaniedbywaną listę definicji:
Kod
<dl>
<dt><a href="www.xxx.zz">www.olddomains.net</a> -<span>Expired Domains Names</span></dt>
<dd>Find expired domains, no longer in whois register.</dd>
<dd>Find valuable domain names with history for cheap price.</dd>
<dd>All expired domains are browsable based on Alexa rank, WayBackMachine, google pagerank and many more.</dd>
<dd>You can also check their previous content, backlinks, most used keywords and meta data values.</dd></dl>
Kod
<h1>Why should You try <b>expired domain names</b> ?</h1>
znacznik <b> to raczej kosmetyka, jednak służy warstwie prezentacyjnej, może lepiej strong?
mozna trochę tę treść urozmaicić np. <acronym> dla seo itp itd
niesemantyczna nazwa klasy:
Kod
<div class="black_text">
style w kodzie HTML:
Kod
<div class="domain_text" style="margin-bottom:10px;">
brak label dla formularza, w ogóle ciut ciut za dużo divów i leciutki classitis.
Od strony CSS, to najlepiej moim zdaniem dać dla body wartość small, a nie w px, fonty w px sux:
Kod
font-size:small;
a potem skalować w procentach. Ie6 będzie za to wdzięczny...
Kod
font-family: Arial, Tahoma;
daj jeszcze na końcu sans-serif bo linux nie zajarzy

, ale nie tylko dlatego.
A jak byś chciał już mieć mega semantycznie, to dodaj onclicki itp. za pomocą narzędzi DOM
To tak na pierwszy strzał. Graficznie moim zdaniem ładnie i przyjemnie.
no to pzdr;)