Forum PHP.pl

http://www.456bereastreet.com/archive/2004..._form_controls/

Ogólne wrażenie:
1. Popracować nad kolorystyką:
2. Popracować nad typografią

Kod z początku wydawał się niczego sobie, choć wypadałoby choć jakies wcięcia stosować dla lepszej czytelności. Tak sobie przeglądałam i patrzę a tu taki kwiat:

[HTML] pobierz, plaintext 
<div id=text2>
<table id=news>
<tr><td id=newstytul>Witam</td></tr>
<tr><td id=newstresc><p>Witam  na hackme.boo.pl. Na razie występują tu tylko zadania związane z bezpieczeństwem stron www, ale w przyszłości zamierzam dodać także zadania matematyczne oraz związane z kryptografią.  A tymczasem zapraszam do rejestracji.</p>
</td></tr></table> 
<br>
</div>
[HTML] pobierz, plaintext 

Nie dość, że jednokolumnowa tabelka (nie wiadomo po co) to jeszcze w divie i w dodatku do zrobienia odstępu użyty znacznik <br>. A fuj...

Poza tym nigdzie nie zauważyłam choćby <h1> i nagle nie z gruszki ni z pietruszki w profilu wyskakuje <h4>

Takie coś:

[HTML] pobierz, plaintext 
Login: tester
<br>
Mail: cos@cos.pl <br>
 
Punkty: 5 <br>
Ranga: user<br>
[HTML] pobierz, plaintext 

To aż się prosi o listę <ul>

Na koniec taka uwaga. Namiętnie i z uporem maniaka pomijasz cudzysłowy w których powinny być wpisane wartości atrybutów znaczników, czasem tylko zamiast nich stosujesz apostrofy.

Hmmm Nie można zalogować się do nierozwiązanych zadań (IMG:http://forum.php.pl/style_emoticons/default/sad.gif)

To może i ja się wypowiem.

Wygląd:
Strona bez życia. Nic mnie w nim nie pociąga. Szarawe tło to bardzo kiepski pomysł. Amatorskie gradienty, za duże bordery. Bardzo kiepsko.
Logotyp nie jest klikalny. Wszędzie ten sam, mdły kolor. Brak wyróżnienia pozycji menu, w której aktualnie się znajdujemy.
Ogólnie: 3/10

Kod html:
Brak jakiejkolwiek tabulacji! Słowa kluczowe za mało mówiące o stronie, a opis jest genialny. Dlaczego nazwy class w divach nie wpisujesz w cudzysłowach? Następnym razem daj do oceny kod nad którym pracowałeś więcej niż godzinkę.
Ogólnie: 2/10.

Kod css:
Brak tabulacji!

[HTML] pobierz, plaintext 
font-family: "Trebuchet MS";
[HTML] pobierz, plaintext 

Używaj bardziej uniwersalnych czcionek, a jeżeli już musisz taką podać, to jako drugą w kolejności wpisz uniwersalną. Nie ujednolicasz sobie pisania kodu css. Zauważyłem, że czasami na początku dasz padding-i, a czasami font-y. Musisz zastosować hierarchię. Wtedy będzie Ci się łatwiej edytowało kod.

[HTML] pobierz, plaintext 
border-style: solid;
border-width: 2px;
border-color: #337de1;
[HTML] pobierz, plaintext 

Takie coś robi się za pomocą samego border-a.

Działanie:
Po zalogowaniu się muszę kliknąć "Dalej". Czy nie lepiej od razu odświeżyć stronę? Chciałem zobaczyć o co chodzi z tą stroną, ale w pierwszym zadaniu wyświetla brak pliku.

Ten post edytował Kildyt 13.12.2008, 12:31:10