Witam!

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

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

Pozdrawiam i dziękuję...

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

nom, spoko Twoje zdanie...

czekam na dalsze opinie.

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ą.

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

jak dla mnie 7/10

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

7/10

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

Pozdrawiam i czekam na wiecej

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

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

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

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

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!

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

no, mam nadzieje ze dobrze zabezpieczone


czym to grozi to ukazanie sesji?


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

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

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

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

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.

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?

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

Yyyyyyy Boje się

A tak serio! Czy dobrze jest to zabezpieczone?

użyłem strip_tags(), funkcja usuwa mi tag <script>

a informacje zaczerpnąłem w pierwszej listowanej stronie w google:
http://www.gajdaw.pl/varia/xss.html

Napisz coś więcej na temat tego jaki efekt chcesz osiągnąć w XHTML i CSS.

aj... juz sporo zamieszałem w temacie i znowu rozwiązałem problem. Mianowicie dodałem do diva który ma byc kolumną po prawej wartośc float:right a w kolumnie po lewej nic! Udało się!

Natomiast następny problem mam!
Mianowicie, chce na Divach zrobic menu poziome:

| Div 1 | Div 2 | Div 3 | Div 4 |

motyw jest:

[HTML] pobierz, plaintext 
#menu li {display:table-cell;}
 Elementy listy w menu zostaną ułożone poziomo
[HTML] pobierz, plaintext 

niestety nie na IE ;|

menu nie na divach tylko na liście nie uporządkowanej (ul).

Są dwie drogi:

[HTML] pobierz, plaintext 
li {display: inline}
[HTML] pobierz, plaintext 

lub

[HTML] pobierz, plaintext 
li {float: left}
[HTML] pobierz, plaintext 

ten drugi sposób daje Ci możliwość umieszczenia wewnątrz li elementu ze zdefiniowanym display: block. Np.:

[HTML] pobierz, plaintext 
li {float: left}
li a {display: block; width: 100px; height: 100px}
[HTML] pobierz, plaintext