začiatočník

Já osobně myslím, že pro začátek je to dobrý.

Jen to menu je určitě lepší textové než formou obrázku (respektive textové a obrázek na pozadí) a blikací lištu bych změnil na neblikací.

Jinak je samozřejmě co vylepšovat, tak s chutí do toho!

já bych se na obrázky úplně vykašlal a doporučil to udělat komplet přes css

http://www.jakpsatweb.cz/priklady/jakotlacitko.html

jinak abys těm tlačítkům mohl nastavit v css šířku a výšku tak musíš v css dodat zápis display:block

U toho menu by nebylo špatné, kdyby vše bylo stejně velkým písmem. To ale vyřešíš, pokud ho uděláš tak, jak psal Ikarus.
Za neblikací lištu také hlasuji. Jednak to je děs, jak tam stále něco bliká, druhak to nemá žádný význam (nebo jo?).

Miro
Javascript ale přece vůbec nepotřebuješ. Při troše snahy dosáhneš pomocí CSS stejného (resp. velmi podobného) efektu. A když už trváš na obrázku na pozadí, i to lze pomocí CSS:

a {display:block; width:šířka; height:výška; background-image:url('cesta_k_tlacitku');}
a:hover {background-image:url('cesta_k_aktivnimu_tlacitku');}

Jinak jako začátečníkovi ti doporučuju na javascript pokud možno na chvíli zapomenout - nejdříve se pořádně nauč CSS. JavaScript má tu "úžasnou" vlastnost, že začátečníky táhne k využívání nestandardních řešení a posléze vede ke kýčovitosti celých stránek (i u mě to tak bylo:-)))