Ocena kobiecego kodowania, design + kodowanie do oceny Opcje

[Natalka]

Witam

W temacie poniżej pytałam co sądzicie o moim "start-upie", postanowiłam zmienić design i chciałabym prosić w chwili obecnej o ocenę zarówno
wyglądu jak i kodowania.

Do cięcia poprzedniej grafiki się nie przyłożyłam i trochę olałam sprawę, tutaj mam nadzieję że trochę się zrewanżowałam i co nieco nadrobiłam.

Wszelkie porady, co zmienić są mile widziane.

Waga strony: 143kb i to ma bardzo duże znaczenie dla mnie bo strona będzie miała dużą ilość odsłon, dlatego też oszczędzałam na grafice i
starałam się zastąpić na wszelkie możliwe sposoby grafikę w CSS'ie.


Link do strony: http://www.szablonallegro.pl/new/index.html


A tutaj kolejne podstrony nie zakodowane jeszcze:

Strona z widokiem na szablon

Kontakt

Pomoc

Dodatki

logowanie + w górnym prawym rogu widok po zalogowaniu



Z góry dziękuje wszystkim za pomoc (IMG:style_emoticons/default/smile.gif)

Stronę sprawdzałam tylko w Mozilli i IE7, więc jak ktoś sprawdzi i wszystko będzie w porządku lub nie to proszę o informacje.

Ten post edytował Natalka 15.02.2010, 03:46:32

[Kildyt]

Design średniej jakości, nie zaskakuje, ale również nie robi obciachu.

Kod

background-color:#E1E1E1; background-image: url(images/bg.jpg); background-repeat: repeat-x;

wszystko do background.

H1 kichę warte. Pusty link dla google ma bardzo niską jakość.
Linki do rejestracji i logowania na ul.
Title w linku to rozwinięcie anchora, a nie powtarzanie co jest zupełnie bezsensu.

[HTML] pobierz, plaintext 
<p class="online">Aktualnie online: <span>15</span></p> 
[HTML] pobierz, plaintext 

[HTML] pobierz, plaintext 
<p class="nawigacja">Tutaj jesteś: <a href="#" title="Strona główna">Strona główna</a></p> 
[HTML] pobierz, plaintext 

[HTML] pobierz, plaintext 
<p>Nazwa: <a href="#" title="Szablon kosmetyczny">Szablon kosmetyczny</a></p> 
<p>Cena: <span>30,00 pln</span></p>
[HTML] pobierz, plaintext 

[HTML] pobierz, plaintext 
<p class="copyright">Copyright ˆ 2010 <a href="#" title="Szablon allegro">SzablonAllegro.pl</a></p>
[HTML] pobierz, plaintext 

To nie jest paragraf.

[HTML] pobierz, plaintext 
<p class="wyszukiwarka">Wyszukiwarka szablonów:</p> <form action="#" method="post"><input type="text" name="wyszukiwarka" id="wyszukiwarka" /> <input type="submit" name="wyszukaj" id="szukaj" value="Szukaj" /></form>
[HTML] pobierz, plaintext 

"Wyszukiwarka szablonów" powinien być labelem. Dodaj fieldset i strict'a masz z głowy.
Angielskie nazewnictwo to oznaka profesjonalizmu i użyteczności.

[HTML] pobierz, plaintext 
<a href="#" class="konsultant" title="Konsultant"></a> 
[HTML] pobierz, plaintext 

Tyle samo wart co w h1.
Komentarze dot. budowy thema w dobrze napisanym kodzie są zbędne.
Definiujesz język angielski strony.
Aby uzyskać większą przejrzystość kodu radziłbym zastosować tzw. nazewnictwo wielbłądzie. Dla przykładu: "szablonboxyellow" to "szablonboxYellow".

Następnym razem nie zaznaczaj, że kodowanie jest "kobiece" bo co niektórzy mogą patrzeć na ciebie łagodniejszym okiem, a to niewskazane. (IMG:style_emoticons/default/winksmiley.jpg)