[www]Strona szkoły Opcje

[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ę.

Ten post edytował ziom 8.03.2007, 07:55:13

[kallosz]

jak dla mnie jest OK. Newsy widze ze kozystaja z AJAX... (fajna koncepcja), stronka bardzo przejzysta... (IMG:http://forum.php.pl/style_emoticons/default/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]

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ę

@Zajec: zapomniałeś o inwazyjnym JavaScript (IMG:http://forum.php.pl/style_emoticons/default/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ą (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

pozdr.

Poprawie, faktycznie zapomniałem o tym ;]

Ten post edytował ziom 8.03.2007, 10:45:13