Ostatnio dostałem zlecenie na, że tak powiem "odwzrorowanie" serwisu bajer.pl z dodatkami Ajax. Podjąłem się wyzwania. Cały kod php jest opatry na Vframe framework, jestem autorem grafiki (oprócz loga) oraz kodu php. Nie jestem odpowiedzialny za treści strony, bowiem to nie moja działka.

Zapraszam do oceny wykonania mojej pracy

www.fotka4u.com

Pozdrawiam, Athlan

Wszystie listy odnośników powinny być zrobione za pomocą list, czyli <ul>, <li>.

Używasz XHTML 1.1, a stosujesz tak przestarzałe znaczniki jak font.

Jeśli chcesz pisać w XHTML 1.1, to wysyła przeglądarkom stronę jako dokument XML, czyli z mime-type application/xhtml+xml. Takie text/html to zostaw dla badzIEwia. http://doktorno.boo.pl/index.php?q=art008

Podstawowym podziałem elementów XHTML jest podział na liniowe i blokowe. Blokowych *nie wolno* umieszczać wewnątrz liniowych, tak jak to robisz z wrzucaniem div do a. Jeśli chcesz mieć blokowy odnośnik to zrób a { display: block; }


Dodano: jeszcze jedna uwaga ─ stosujesz komentarze XML <!-- wewnątrz script. Wiesz, że przeglądarka ma prawo wyrzucić cały skrypt? Pozbądź sie tych komentarzy.

Design:

- Kolorystyka jest za rozowa

- Efekt "szkla" w pasku nawigacyjnym na gorze jest zle zrobiony

- Te malutkie teksty pod linkami w nawigacji w topie robia syf -> robi sie napackakane

- Kolory na dole ( na rozowym pasku sa nie widoczne)



Usability:

- Mysle ze lepiej byloby w glosowaniu zrobic linki zamiast przycikow i zeby nie wyskakiwol dialog o zalogowaniu tylko przekierunkowywal na strone z logowaniem.





Pozdrawiam

Zrobiłem i troszkę się sypie, przykład:

Podświetla błąd składni: </a>. A to Ci żarcik? Za takie coś ja dziękuję chyba że wina leży po mojej stronie

Font uzył mój zleceniodawca, pinformowałem mu o <span> , dzięki za info

Odnośnie elementów blokowych: w stylu jest display: block; dla elementu div.menu a.button. czyli chyba dobrze nie bloki w blokach

Na temat komentarzy: mam użyć zapisu :
? czy może wogóle wywalić jakiekolwiek komentarze i naskrobać czysty kod?

Teraz pare zdań do @nasty_psycho

kolorystyka, wielkość fontu i efekt szkła to kwesia gustu, tak chciał zleceniodawca - tak zrobiłem Ale masz rację, lecz nie ode mnie to zależy Miałeś oceniać to oceniasz - słusznie

About usablity: wiem, powinienem, ale nie mam w tym projekcie takiego czegoś jal strona logowania. Form jest po prawej stronie.

Zapraszam do dlaszej oceny, dziękuje za dotychczasowe głębokie przejżenie kodu

Pozdrawiam, Athlan

1) Wina po Twojej stronie. Próbujesz zamykać znacznik a, którego wcześniej nie otworzyłeś - więc wyrzuca błąd.

2) <![CDATA[ warto stosować, nie spowoduje błędów gdy użyjesz później znaków mniejszości, większości, amperandół itp. w skrypcie.

A to ciekawe, przyjżyj się temu kawałkowi kodu:


znacznik jest otwarty, dla bonussu znalazło się i jego zamknięce

dalej wina po mojej stronie?

Tak, po twojej ( )
<a href="http://fotka4u.com/info/regulamin/" title="" />akceptujesz nasz regulamin</a>
powinno być:

[XML] pobierz, plaintext 
<a href="http://fotka4u.com/info/regulamin/" title="</span>">akceptujesz nasz regulamin<<span style='color:blue'>/a>
[XML] pobierz, plaintext 

A strona pod względem oprawy graficznej wogóle mi się nie podoba, mogłeś bardziej poszaleć z tym różem ;-)

-logo, do wymiany. jest trochę takie, hmmm, kiepskie. Czcionka w nim jest tragiczna .
-menu, do wymiany jak już ktoś napisał efekt odbicia raczej kiepski

-reszta, jak poprawiony będzie top to będzie OK

Pozdrawiam