Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [www]Strona szkoły
Forum PHP.pl > Inne > Oceny
ziom
http://www.norwid.tychy.pl/index.php
Na wstępie zaznaczę, że to stronicowanie też wydaje mi się kiepskim pomysłem ;]
Proszę o ocenę.
kallosz
jak dla mnie jest OK. Newsy widze ze kozystaja z AJAX... (fajna koncepcja), stronka bardzo przejzysta... smile.gif
Zajec
Uwagi (negatywne):

1) Doctype: http://pornel.net/transitional
2) Używasz przestarzałych znaczników center i font: http://www.norwid.tychy.pl/reg.js
3) Niepotrzebnie moim zdaniem korzystasz z elementów graficznych, które mógłbyś zastąpić tekstem: http://www.norwid.tychy.pl/images/katalog.gif
4) Brak atrybutów alt opisujących obrazek. Przykład: <img src="images/katalog.gif" alt="" /> Brakuje tu alt="Menu".
5) Do zbudowania menu (czyli listy odnośników) nie wykorzystałeś listy (ul) z odnośnikami (a). Zła semantyka
6) To samo co wyżej, ale z nagłówkami. Zamiast zrobić nagłówki w <hx> korzystasz z p, strong itp.
7) Kodowanie strony: windows-1250 jest tragiczne, ale mimo wszystko obsługuje polskie literki. Po co więc używać kwiatków typu planuje się otworzy&#263
8) Zdecyduj się co chcesz osiągnąć w stopce. Widzę, że coś kombinujesz z twardymi spacjami, ale między nimi masz też zwykłe.
9) Wg specyfikacji XML (a XHTML jest oparty o XML) znaczniki muszą być pisane małymi literami. Popraw to np. na stronie http://www.norwid.tychy.pl/artykul-153-search.html
10) Zapis typu: <a href="java script:history.back(1)"><< Wstecz </a> jest niedozwolony. Znaki mniejszości, większości są zarezerwowane dla znaczników. Jeśli chcesz je umieścić na stronie, skorzystaj z zapisuje &lt; (od less than) oraz &gt; (greater than)

Na pocieszenie dodam, że jestem chyba najbardziej czepiającym się kodu człowiekem na tym forum ;-)
siemakuba
@Zajec: zapomniałeś o inwazyjnym JavaScript :)

@ziom: przy wyłączonym JS nie będę mógł użyć stronicowania ani oglądać powiększeń zdjęć w galerii (o tym jak zrobić dobre popupy czytaj tu: http://pornel.net/pups/). Co do stronicowania - najpierw oprogramuj ten mini formularz tak, aby działał bez żadnego JS. Potem przechwytuj zdarzenia i jeże ktoś ma JS serwuj mu wersję AJAXową :)

pozdr.
ziom
Cytat(Zajec @ 8.03.2007, 09:18:57 ) *
Uwagi (negatywne):

1) Doctype: http://pornel.net/transitional
2) Używasz przestarzałych znaczników center i font: http://www.norwid.tychy.pl/reg.js
3) Niepotrzebnie moim zdaniem korzystasz z elementów graficznych, które mógłbyś zastąpić tekstem: http://www.norwid.tychy.pl/images/katalog.gif
4) Brak atrybutów alt opisujących obrazek. Przykład: <img src="images/katalog.gif" alt="" /> Brakuje tu alt="Menu".
5) Do zbudowania menu (czyli listy odnośników) nie wykorzystałeś listy (ul) z odnośnikami (a). Zła semantyka
6) To samo co wyżej, ale z nagłówkami. Zamiast zrobić nagłówki w <hx> korzystasz z p, strong itp.
7) Kodowanie strony: windows-1250 jest tragiczne, ale mimo wszystko obsługuje polskie literki. Po co więc używać kwiatków typu planuje się otworzy&#263
8) Zdecyduj się co chcesz osiągnąć w stopce. Widzę, że coś kombinujesz z twardymi spacjami, ale między nimi masz też zwykłe.
9) Wg specyfikacji XML (a XHTML jest oparty o XML) znaczniki muszą być pisane małymi literami. Popraw to np. na stronie http://www.norwid.tychy.pl/artykul-153-search.html
10) Zapis typu: <a href="java script:history.back(1)"><< Wstecz </a> jest niedozwolony. Znaki mniejszości, większości są zarezerwowane dla znaczników. Jeśli chcesz je umieścić na stronie, skorzystaj z zapisuje &lt; (od less than) oraz &gt; (greater than)

Na pocieszenie dodam, że jestem chyba najbardziej czepiającym się kodu człowiekem na tym forum ;-)

Muszę przyznać, że liczyłem trochę na to, że się przyczepisz, będzie trzeba zabrać się za XHTML'a, a z tymi znacznikami w artykułach... muszę napomnieć zespół strony. ;]
Zaznaczę, że galeria to szczyt 'frajerstwa', ale to się poprawi. Musiałem wiedzieć na co zwracać uwagę

Cytat(siemakuba @ 8.03.2007, 10:10:37 ) *
@Zajec: zapomniałeś o inwazyjnym JavaScript smile.gif

@ziom: przy wyłączonym JS nie będę mógł użyć stronicowania ani oglądać powiększeń zdjęć w galerii (o tym jak zrobić dobre popupy czytaj tu: http://pornel.net/pups/). Co do stronicowania - najpierw oprogramuj ten mini formularz tak, aby działał bez żadnego JS. Potem przechwytuj zdarzenia i jeże ktoś ma JS serwuj mu wersję AJAXową smile.gif

pozdr.

Poprawie, faktycznie zapomniałem o tym ;]
To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.
Invision Power Board © 2001-2025 Invision Power Services, Inc.