www.krotex.net, proszę o ocenę Opcje

[gron]

Ostatnio chodził mi długo pomysł po głowie i go zrealizowałem w końcu, co myślicie o tej stronie?
Krotex.net Jest to strona mająca za zadanie reklamować aukcje z Allegro, ale co ja tu będę (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Ten post edytował gron 6.06.2008, 21:47:19

[AjaxSrajax]

No to jadziem (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

<div id="top-nav"> - niepotrzebne, zamiast tego id="top-nav" dla <ul> ***.
znaczniki <b> nie są fajne, zamiast tego CSS.
<div id="nav"> - niepotrzebne, zamiast tego id="nav" dla <ul> ***
<div class="clear"> </div> - takie clearowanie nie ma sensu; zamiast tego clear dla pierwszego blokowego w kolejności.

Kod

<div id="visit">
<h2>Jak zacząć?</h2>
<p>1. Zakładasz konto na Krotex.net (Potem zatwierdzasz i logujesz.)</p>
<p>2. Dodajesz użytkownika lub kilku z Allegro. (Z tym systemem w tej chwili współpracujemy.)</p>
<p>3. Na forach internetowych lub blogach w "podpis" lub "sygnaturkę" dodajesz skrypt który generuje Krotex.net (Tą część wykonujesz tylko raz. Możesz też dodawać swój podpis w dowolnych miejscach.)</p>
<p>4. I to wszystko. Od tej chwili zawsze gdy sprzedajesz coś przez Allegro, twoje aukcje widzi wiele osób.</p>
</div>

Tu aż się prosi pozbyć diva i użyć listy uporządkowanej <ol>

To mi też wygląda na listę, tym razem <ul>:

Kod

<p>
                                       zestaw_sam,
                                                    PlayForFan,
                                                    Hazard,
                                                    Chardy1975,
                                                    <a target="_blank" href="?id=37">demo</a>,
                                                    Mazak,
                                                    Akira,
                                                    info,
                                                    stopspamu,
                                                    nowy,
                                                    Centkowany,
                                                    fuhflesz,
                                                    modna-kasia,
                                                    blach0r,
                                                    Patosław,
                                                    BLUE,
                                                    Ma_niek,
                                                    Eltona,
                                                    Rafał,
                                                    gronsoft,
                                                    mała,
                                                    szybki,
                                                    ka$a,
                                                    maxim,
                                                    <a target="_blank" href="?id=14">konsole</a>,
                                </p>

brak labeli dla forma - nieładnie...
To tak z pierwszej strony. W porównaniu z kodem innych nienajgorzej, ale daleko do ideału. Za dużo divów ciut ciut (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

*** może jestem maniakiem, ale dałbym troszkę bardziej semantyczne nazwy klas oraz id.

pozdro (IMG:http://forum.php.pl/style_emoticons/default/haha.gif)