Hackme, Prosze o ocene mojej strony Opcje

[maniekl2]

http://hackme.boo.pl

Proszę o ocenę mojej strony:
1) layout
2) treść (czyli głownie zadania)
3) bezpieczeństwo
4) Ogólne wrażenie

[ShadowD]

Na tło dal bym jakieś kreseczki bo przy ~1600/1000 monotonne jest strasznie, napis na logu zlewa się z nim.

Dodatkowo powinieneś ostawić jakieś testowe konto by nie trzeba było się rejestrować...

Ps Dwa błędy w html wypadało by poprawić... (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Ten post edytował ShadowD 9.12.2008, 23:57:43

[erix]

4) Ogólne wrażenie

Pierwsze wrażenie? "Coś pusto tu".

1) layout

Schludny aż do przesady. (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

2) treść (czyli głownie zadania)

:

Dodatkowo powinieneś ostawić jakieś testowe konto by nie trzeba było się rejestrować...

[maniekl2]

dodałem konto testowe i strona validuje się poprawnie

Login: tester
Hasło: test

Ten post edytował maniekl2 10.12.2008, 18:10:50

[timon27]

"<form>" też da się css-ować!
zrób to, gdyż na Twej stronie są inputy ze standardowym wyglądem i nie wygląda to dobrze.
Dodatkowo gdyby w tym górnym panelu były powoli spadające literki zamiast statycznego obrazu byłoby zaje..fajnie.

I za dużo bieli - gdzieś musisz dodać tło - w background może być ciemniejsze, w reszcie np. jasny błękit

Ten post edytował timon27 11.12.2008, 23:15:28

[lord_t]

@timonie27 owszem da się, ale to nie ma sensu, bo każda przeglądarka i tak pokaże to inaczej, że już nie wspomnę o niepoddawaniu się na style inputa do pobierania plików. (Widziałem gdzieś w necie stronę, na której były pokazane różne inputy z różnych przeglądarek, chętnie bym Cię tam odesłał, ale niestety nie znam adresu [może ktoś zna?]).

[erix]

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

[maniekl2]

Błękitny kolor nie pasował, więc na tło dodałem lekką szarość. Były chwilowe problemy z testerem bo ktoś zmienił hasło (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) ale już jest wszystko w porządku. Można się logować:
Login: tester
Hasło : test

erix
Dzięki za link

Ten post edytował maniekl2 12.12.2008, 14:07:40

[JoShiMa]

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)

[maniekl2]

Dzięki za rady. Postaram się to w najbliższym czasie poprawić.

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

Nie rozumiem dokładnie o co Ci chodzi. Mógłbyś to rozwinąć ?

[JoShiMa]

Nie rozumiem dokładnie o co Ci chodzi. Mógłbyś to rozwinąć ?

Mogłabym

Otwieram listę zadań. Jest tylko jedna kategoria, WWW. Klikam na nią wyświetla się lista zadań. Klikam w zadanie pojawia się okno zachęcające do wpisania hasła. Wpisuję hasło i dowiaduję się, że jest nieprawidłowe i niniejszym nie mogę obejrzeć zadania.

[maniekl2]

Chyba źle zrozumiałaś po co jest tam to hasło.
Nad inputem do wpisania hasła jest link do zadania. Gdy rozwiążesz zadanie otrzymasz hasło, które musisz wpisać właśnie do tego inputa. Gdy to zrobisz otrzymasz punkty.

Wszystkie kwiatki poprawiłem (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Ten post edytował maniekl2 12.12.2008, 20:00:58

[lord_t]

Przejrzyj wszystkie linki które wysyłasz na mejla. Bo ja dostałem raz coś takiego na początku linku:http://zadania.skate.boo.pl/

Przy okazji w tytule zjadło polską literę.

[maniekl2]

Nie wiem jak to możliwe że taki link dostałeś. Pokaż dokładnie jak wyglądała wiadomość.

[lord_t]

KLik : http://zadania.skate.boo.pl/zadanie12/haslo.html

[maniekl2]

Dzięki. Przenosiłem z innego serwera i dlatego wystąpił taki błąd.

[Kildyt]

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

[maniekl2]

Przyznaję że nie potrafię robić layoutów. Nie mam daru do łączenia kolorów dlatego i tak jestem zadowolony z tego co stworzyłem.
Ale dzięki za krytykę. Na pewno da mi to coś do myślenia. Opisy, kod są jakie są, bo nie przykładałem się do nich. Bardziej myślałem nad kodem php. Ale w najbliższym czasie popracuję nad tym kodem html i css.

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.

Zastanawiałem się nad tym odświeżaniem strony, ale wybrałem link "Dalej". Możliwe że to zmienię.

Przepraszam za to pierwsze zadanie. Wczoraj niechcący usunąłem jeden plik. Już jest wszystko w porządku.

Ten post edytował maniekl2 13.12.2008, 10:29:37

[AjaxSrajax]

Kod css:
Brak tabulacji!

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

Używaj bardziej uniwersalnych czcionek, a jeżeli już musisz taką podać, to jako drugą w kolejności wpisz uniwersalną.

Trebuchet MS nie jest uniwersalny wg. Ciebie? To jedna z najczęściej używanych czcionek dzisiaj. Ale fakt, że wers CSS powinien brzmieć:

Kod

font-family: "Trebuchet MS", sans-serif;

Dodanie 'sans-serif' chyba wiadomo co spowoduje;)

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.

Ciekawa teoria, ale chyba niewykonalna. Z tego względu, że pisząc kod CSS zwłaszcza pod windowsem testujesz, sprawdzasz dodajesz i odejmujesz wersy i patrzysz jaki jest efekt. Ciężko zachować tę samą kolejność, zresztą kod CSS jest tak prosty, że nie ma tam za bardzo co analizować (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

[Kildyt]

Trebuchet MS nie jest uniwersalny wg. Ciebie? To jedna z najczęściej używanych czcionek dzisiaj.

Gdzie ty żyjesz? (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) Dwie magiczne litery na końcu nazwy fonta mówią wszystko. (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Sprawdziłem na ubuntu 8.10 i takiej czcionki nie ma, a paczki z windowsowskimi czcionkami jak na razie nie zainstalowałem.

Ciekawa teoria, ale chyba niewykonalna. Z tego względu, że pisząc kod CSS zwłaszcza pod windowsem testujesz, sprawdzasz dodajesz i odejmujesz wersy i patrzysz jaki jest efekt. Ciężko zachować tę samą kolejność, zresztą kod CSS jest tak prosty, że nie ma tam za bardzo co analizować (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Jak to niewykonalna? (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) Ja jakoś sobie radzę i bardzo ułatwia mi to pracę. Mam taki system (od góry): definiowanie bloku, font-y, rozmiary, background-y, border-y, padding-i, margin-y. Te 3 ostatnie kiedyś mieszałem, ale ostatnio trzymam się zasady.