http://stalowemiasto.pl

prosze o ocene

Czy się mylę, czy kiedyś dawałeś top do oceny?

- rozwijane menu nie jest dokładnie pod paskiem menu,
- stat4u w "inne" (polecam google analytics),
- napisy w topie masz w dwóch plikach (zwykły i ten po najechaniu); radziłbym zamieścić je w jednym pliku, żeby nie było efektu wczytywania obrazka (przy najechaniu położenie zdefiniujesz w css),
- pliki zapisane w .png strasznie dużo ważą, więc na strony się raczej nie andają (chyba, że ikonki, które wymagają przezroczystości),
- moim zdaniem strona jest zbyt wąska - lewe menu jest szerokie, a pole na tekst wygląda jakby większe menu,
- w tekście nie ma obramowania dla obrazków, przez co litery są zaraz koło obrazka,
- "Pogoda" - lepiej gdybyś pod obrazkiem napisał co oznacza ta chmurka/słonko,
- "Twoja ocena:" - po najechaniu na cyfrę jej tło z szarego staje się białym, przez co zlewa się z tłem strony - moim zdaniem jest to zbyt dobry pomysł,
- po co wyświetlasz formularz dodawania komentarza, gdy i tak go nie mogę dodać, ponieważ nie jestem zalogowany?,
- w czacie psują się polskie znaki,
- brak nagłówków w kodzie (h1, h2 itd.),
- za dużo stylów w pliku html,
- polecam UTF-8 do kodowania znaków.

Witam, jeśli chodzi o mnie postanowiłem sprawdzić tą stronę pod kątem bezpieczeństwa. Nie wypowiadam się co do wyglądu, bo nie znam się na tym. Co do bezpieczeństwa to jakby nie patrzeć ma ono niski poziom. Na stronie jest błąd pozwalający na wstrzyknięcie zapytania do bazy, czyli SQL Injection. Wynika on z faktu, iż jedna ze zmiennych nie jest filtrowana.

Tutaj jest błąd:


Tutaj screen, który pokazuje możliwość wyciągnięcia z tego danych:


BTW. Oprócz filtrowania zmiennych radzę zabezpieczyć hasła za pomocą, np. md5 czy sha1, a nie zostawiać je w czystej postaci Elo

Takie konstrukcje:

[HTML] pobierz, plaintext 
<div style="width:206px;height:90px;float:left;background-image:url('././theme/normal/top/bg_topL.png')">
 
        <div style="position:absolute;top:27px;left:11px"><a href="http://stalowemiasto.pl/podstrony/podstrony.php?mode=poczta"><img onMouseOver="zmiana1('poczta')" onMouseOut="zmiana2('poczta')" src="././theme/normal/top/poczta.png" alt="poczta" /></a></div>
        <div style="position:absolute;top:56px;left:11px"><a href="http://stalowemiasto.pl/podstrony/podstrony.php?mode=poczta"><img onMouseOver="zmiana1('poczta')" onMouseOut="zmiana2('poczta')" id="poczta" src="././theme/normal/top/poczta_n.png" alt="poczta" /></a></div>
 
        <div style="position:absolute;top:27px;left:75px"><a href="http://stalowemiasto.pl/szukaj/szukaj.php"><img onMouseOver="zmiana1('szukaj')" onMouseOut="zmiana2('szukaj')" src="././theme/normal/top/szukaj.png" alt="wyszukiwarka" /></a></div>
        <div style="position:absolute;top:56px;left:75px"><a href="http://stalowemiasto.pl/szukaj/szukaj.php"><img onMouseOver="zmiana1('szukaj')" onMouseOut="zmiana2('szukaj')" id="szukaj" src="././theme/normal/top/szukaj_n.png" alt="wyszukiwarka" /></a></div>
 
        <div style="position:absolute;top:27px;left:144px"><a href="http://stalowemiasto.pl/forum/forum.php"><img onMouseOver="zmiana1('forum')" onMouseOut="zmiana2('forum')" src="././theme/normal/top/forum.png" alt="forum" /></a></div>
        <div style="position:absolute;top:56px;left:144px"><a href="http://stalowemiasto.pl/forum/forum.php"><img onMouseOver="zmiana1('forum')" onMouseOut="zmiana2('forum')" id="forum" src="././theme/normal/top/forum_n.png" alt="forum" /></a></div>
 
    </div>
[HTML] pobierz, plaintext 

Robi się za pomoca list a nie za pomocą divów

Początek był obiecujący, ale potem zaatakowały mnie tabele wykorzystane niezgodnie z przeznaczeniem, czyli do stworzenia konstrukcji strony. Blech...

top już kiedyś komentowaliśmy

design mi się podoba - bez większych uwag (może ta pora na to wpływa )

ale jest jedna bardzo istotna rzecz - strona jest za duża na pod względem wysokości (za długa). Trzeba się nieźle "nascrollować" żeby dotrzeć do końca. Strony nie powinny być dłuższe niż 1,5 - 2 ekrany. Radzę wprowadzić stronicowanie.

onet ma ponad 5 ekranow

ale to zasyfiony portal z mnóstwem przeróżnych informacji - strona miasta odbiega od takiej koncepcji

Ma, ale dobrym zwyczajem jest wzorować się na najlepszych.

no to chyba ten "zasyfiony" onet jest najlepszy ?

nie, nie jest

Poza tym porównaj ilość informacji zawartych na onet a ilość informacji na Twojej stronie - różnica diametralna.