[www][layout] Strona Gimnazjum, zaktualizowane - strona juz istnieje :) Opcje

[k0b3]

Witam!

Prosze o ocene i ewentualne poprawki. Co zmienic, co moze dodac?

http://img48.imageshack.us/img48/8002/forumphpvy7.jpg

Pozdrawiam i dziękuję...

Ten post edytował kobemat 3.05.2007, 17:32:27

[Daimos]

jak dla mnie tak nawalone za bardzo w typ topie jest
mialem chec zdrapac polowe tych rzeczy z monitora
kwadraty w topie, linia przy "strona glowna", czy menu - jakos kojarzy mi sie bardziej z prezentacja power pointa, niz z jakas strona www
ramka na dole (liczba osob online) - przesadzone cienie
sory za krytyke, no ale nie podoba mi sie za bardzo (moj gust, moja opinia, czekaj na inne)
daje
2/10

[k0b3]

nom, spoko Twoje zdanie...

czekam na dalsze opinie.

[starach]

W mojej opinii bardzo ładny lej.
( ale nie sugeruj się bo mam koszmarny gust )
Tylko nagłówek mógłby byś jaskrawszy i dwójka nie w tym kolorze ewentualnie inną czcionką dwójka,
a kolor ten sam co nagłówek jeszcze by dobrze było żebyś zmienił gradient na podświetlanie przycisków
bo mam dziwne wrażenie że te kolory się gryzą.

[frickle]

Rzeczywiście, top wydaje się być trochę przeładowany. Ale ogólnie mnie się podoba.

jak dla mnie 7/10

[remik09]

jak na strone gimnazjum to lay calkiem w porzadku, wiekszosc szkol ma laye dosyc podobnie robione wiec nie ma zle

7/10

[k0b3]

Dziękuje za komentarze za te zle i oczywiscie dobre.

Pozdrawiam i czekam na wiecej

[Jabol]

Fajne. Żywe i przejrzyste (pomimo, że rzeczywiście trochę przeładowane)

[k0b3]

OK! Strona z layoutu już istnieje
Oczywiście nie jest jeszcze oficjalnie otwarta. Proszę o dalsze opinie grafiki (choc już było o niej powiedziane), kodu oraz funkcjonalności strony!

zapraszam na:
http://2gimn-kielce.edu.pl/main.php

a za drzwiami wygląda tak





Pozdrawiam

[babejsza]

Jeżeli chodzi o kod to ... ojojoj

1. Tabelki, tabeleczki
2. CSS inline
3. iso-8859-2 i takie kwiatki: &#347 &#360
4. XHTML 1.1? - gdzie ?
5. JS w kodzie strony
6. menu nie w liście

To tyle

Ten post edytował babejsza 3.05.2007, 20:40:03

[envp]

Czesio lubi ziemniaki

http://2gimn-kielce.edu.pl/print.php?klasa...e)%3C/script%3E

smakowe tymbardziej:
http://2gimn-kielce.edu.pl/main.php?page=p...=../panel/index

Pozdrówki

Ten post edytował envp 4.05.2007, 00:17:11

[k0b3]

Ouuu mowisz "Czesio lubi ziemniaki "

zaznaczam ze stawiam kroki w phpie a co dopiero w zabezpieczeniach!

czy wystarczy to?

[PHP] pobierz, plaintext 
<?php
$menub = preg_replace( '%/%', '',$_GET['page']);
 
$submenub = preg_replace( '%/%', '',$_GET['sub']);
 
addslashes($menub);
addslashes($submenub);
?>
[PHP] pobierz, plaintext 

dzieki za pomoc!

[tiraeth]

Temat: Bezpieczenstwo skryptow PHP
Tutaj powinieneś znaleźć wszystko co potrzebne

1) Sprawdzaj czy plik istnieje przed includowaniem go: file_exists" title="Zobacz w manualu PHP" target="_manual
2) Co do wywalania kodu HTML ze zmiennych (np. z ?klasa=) to zainteresuj się strip_tags" title="Zobacz w manualu PHP" target="_manual.

Wracając do oceny strony... jakoś mi się zbytnio nie podoba... słabizna

[k0b3]

no, mam nadzieje ze dobrze zabezpieczone

http://2gimn-kielce.edu.pl/print.php?klasa...e)%3C/script%3E

czym to grozi to ukazanie sesji?

Wracając do oceny strony... jakoś mi się zbytnio nie podoba... słabizna tongue.gif

a no szkoda szkoda
Za dużo tabelek hmmm... Strona w XHTML ma skladac sie tylko z DIVów?

[Turgon]

Nie, ale tabelki do czego innego służą. Służą do prezentowania danych tabelarycznych.

[Jabol]

no, mam nadzieje ze dobrze zabezpieczone
czym to grozi to ukazanie sesji?

W sumie to niczym... ogólnie przecież cookie i tak mogę sobie objrzeć skoro jest na moim kompie.

[tiraeth]

@Jabol: Może i cookie możesz obejrzeć, ale warto zabezpieczać się przed takimi "fortunkami", bo potem może dojść do użycia XSS...

[Kildyt]

Hmm, jak dla mnie design bez wyrazu...
Polecam nie wyświetlania linku do panelu admina dla zwykłych userów. Do tylko pokusa dla hakerów.

Za bardzo kolorowe, to już jest w końcu gimnazjum, a nie podstawówka.
Ogólnie nawet spoko.

[k0b3]

tiraeth dzięki za pomoc w znalezieniu luk! te wyskakujace cookies zabezpieczyłem strip_tags()

właśnie poczyniłem prace żeby oczyścic troche kod z tych tabelek. Moze zbyt poźno, powinienem to zrobic zanim dałem do oceny, ale jeśli mówicie ze to takie ważne w takim razie zabrałem się do roboty cały czas się ucze!

dzieki za ocenki, oczywiście czekam na dalsze komentarze

Pozdrawiam!

/////////////////////////////////////edit

dopisuje jeszcze z nurtującym pytanie nie tylko jednego amatora.
Czy w końcu jest wyjście - możliwość uzyskania tabelki na divach w IE?

nie cierpię tej przeglądarki, ale niestety większość ludzi korzysta z niej!
Czytając w internecie jest wiele dyskusji na tematy nieformalności IE i stosowania Tabelek nie do przedstawiania danych tabelarycznych, a pozycjonowania stron! Wiem że to źle, zaciera kod XHTML, ale jak wielu już powiedziało - tak jest łatwiej

Tylko czemu najbardziej powszechna przeglądarka źle interpretuje CSS i ciężko dostosować wyświetlanie w IE stosując się do standardów w3c.
wkurza mnie to!

Wracając do pytania. Potrzebuje tylko tabelkę z dwoma kolumnami.
wspaniały tut -> http://kurs.browsehappy.pl/CSS/Table, ale niestety nie działa pod IE. Poza tym position: absolute w moim przypadku nie może być!

co powinienem zrobic?

[envp]

no, mam nadzieje ze dobrze zabezpieczone
czym to grozi to ukazanie sesji?

Wyobraź sobie, że nie bornisz się przed XSS, teraz ja w komentarzach daje sobie notkę i robie w niej bla bla bla a w pole email wpisuje "><script>window.location='http://teojastrona/panel/newsy/del.php?id=all'</script>

i co ktos wchodzi i dostaje forbiden, bo to przekierowanie na panel, ale wchodzi ty, który masz zapisane hasło admina i klapa. Wszystkie newsy papa