Forum PHP.pl

Witam, prosze o ocene. Pocielem ten projekt na zlecenie. Chcialbym wiedziec czy widzicie jakies bledy w kodzie albo w cieciu, tak na przyszlosc zebym mogl poprawic.

Aha meta tagi klient chcial dodac sam

ciecie do xhtml

Ten post edytował shovit 20.01.2008, 21:12:47

Ogólnie bardzo fajowo, ładny i elegancki kod. Jedyne co bym zmienił to dodał atrybuty width i height do obrazków. Dlaczego? Gdyż przy problemach z wyświetleniem grafiki stronka nie zatraci swego układu. W miejscu zdjęć pojawi się ramka/puste miejsce o wymiarach konkretnego zdjęcia.
Reasumując: dobra robota.

Ogólnie kod bardzo dobry. Kilka uwag:
-

[HTML] pobierz, plaintext 
lang="en" xml:lang="en">
[HTML] pobierz, plaintext 

- Obrazki img/onas.jpg, img/podroz.jpg, img/chwile.jpg nie są elementami dekoracyjnymi. Zastępują nagłówki więc atrybut Alt nie powinien być pusty. Dodatkowo powinny być bez tła (zapisane w gifie).
- brak nagłówków
- strona sypie sie w w modelu border box.
- w linkach brakuje atrybutów Title.

Ten post edytował woj_tas 20.01.2008, 22:59:00

Kod dobry, wygląd też fajny:) W div header i footer zamiast span uzyj p. W menu, w liscie, pierwszy i ostatni element listy są nie potrzebne.

Ten post edytował Moli 20.01.2008, 23:25:00

Dzieki za te kilka rad, bede wiedzial na przyszlosc szczegolnie o parametrach rozmiarow obrazka bo to celna uwaga, w zleceniu nic juz nie zmienie bo juz wykonane

Jest dobrze.
Jedynie mam kilka uwag do:

1.

[HTML] pobierz, plaintext 
<li><img src="img/menu1.jpg" alt="" /></li>   [...]
<li><img src="img/menu6.jpg" alt="" /></li>
[HTML] pobierz, plaintext 

< tych "pustych" obrazków nie powinno być w tej liście.

2.

[HTML] pobierz, plaintext 
<img src="img/top1.jpg" alt="" />
		<img src="img/top2.jpg" alt="Ania i Tomek - nasze zdjecia" class="foto" />
		<img src="img/top3.jpg" alt="" />
[HTML] pobierz, plaintext 

Powinieneś to zrobić bez takich obrazków, tylko jako tło div'a.

3.

[HTML] pobierz, plaintext 
<img src="img/onas.jpg" class="header" alt=""/>
[HTML] pobierz, plaintext 

Ten i inne podobne obrazki powinny mieć alt z tekstem z obrazka (czyli w typ przypadku alt="O nas")

4.

[HTML] pobierz, plaintext 
<a href="#"><b>czytaj więcej</b></a>
[HTML] pobierz, plaintext 

Takie pogrubione linki chyba lepiej robić w stylach. Nawet kod jest wtedy czytelniejszy...
Możesz poprostu napisać w stylach: " #page a { text-weight:bold } "

5.

[XML] pobierz, plaintext 
#lay {
      text-align: left;
}
[XML] pobierz, plaintext 

Po co to?

Reszta jest na 99% dobrze. Chyba, że coś przeoczyłem?

jeszcze odnośnie uwagi Molie'go: "W div header i footer zamiast span uzyj p" < Span jest w tym przypadku dobrze użyty...Stopka chyba nie jest jednym akapitem tylko po prostu linijką tekstu, podpisem który nie jest właściwie niczym?

Ten post edytował ano 21.01.2008, 22:26:53

Ja jeszcze bym dodał zmiane obrazka w menu po najechaniu myszką. Bez tego wygląda bardzo sztywno i do końca nie wiadomo czy tam jest link.
Ale ogołnie jest b. dobrze. Ładna, przejzysta, intulicyjna strona co cenie. Nie lubię "wodotrysków", i "fajerwerek", a najbardziej tych zrobionych we flashu.

Dzieki za uwagi. Co to tego ze powinienem uzyc tego jako tła divu - klient wymagał aby w kazdym momencie mogl podmienić głowne zdjecie, a w ten sposob moze je łatwo zmieniac.

Ci do tego text-align: left to jest to po to ze w body dalem text-align: center aby IE centrowalo strone no to dla reszty ustawiam juz normalnie czyli do lewej. Chyba dobrze?

po wyjściu z trybu quirks wystarczy samo margin: 0 auto;